Skip to content

Commit 5d19b67

Browse files
committed
chore(codecs): consolidate enum types
1 parent c97d619 commit 5d19b67

File tree

18 files changed

+144
-296
lines changed

18 files changed

+144
-296
lines changed

lib/codecs/src/decoding/format/gelf.rs

+3-2
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,14 @@ use crate::{gelf_fields::*, VALID_FIELD_REGEX};
2626
/// of vector will still work with the new relaxed decoding.
2727
2828
/// Config used to build a `GelfDeserializer`.
29-
#[derive(Debug, Clone, Default, Deserialize, Serialize)]
29+
#[configurable_component]
30+
#[derive(Debug, Clone, Default)]
3031
pub struct GelfDeserializerConfig {
32+
/// GELF-specific decoding options.
3133
#[serde(
3234
default,
3335
skip_serializing_if = "vector_core::serde::skip_serializing_if_default"
3436
)]
35-
/// GELF-specific decoding options.
3637
pub gelf: GelfDeserializerOptions,
3738
}
3839

lib/codecs/src/decoding/format/json.rs

+2-3
Original file line numberDiff line numberDiff line change
@@ -17,14 +17,13 @@ use super::{default_lossy, Deserializer};
1717

1818
/// Config used to build a `JsonDeserializer`.
1919
#[configurable_component]
20-
#[derive(Debug, Clone, PartialEq, Eq, Derivative)]
21-
#[derivative(Default)]
20+
#[derive(Debug, Clone, Default)]
2221
pub struct JsonDeserializerConfig {
22+
/// JSON-specific decoding options.
2323
#[serde(
2424
default,
2525
skip_serializing_if = "vector_core::serde::skip_serializing_if_default"
2626
)]
27-
/// JSON-specific decoding options.
2827
pub json: JsonDeserializerOptions,
2928
}
3029

lib/codecs/src/decoding/format/native_json.rs

+6-2
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
use bytes::Bytes;
22
use derivative::Derivative;
3-
use serde::{Deserialize, Serialize};
43
use smallvec::{smallvec, SmallVec};
54
use vector_config::configurable_component;
65
use vector_core::{config::DataType, event::Event, schema};
@@ -11,9 +10,14 @@ use super::{default_lossy, Deserializer};
1110
use vector_core::config::LogNamespace;
1211

1312
/// Config used to build a `NativeJsonDeserializer`.
14-
#[derive(Debug, Clone, Default, Deserialize, Serialize)]
13+
#[configurable_component]
14+
#[derive(Debug, Clone, Default)]
1515
pub struct NativeJsonDeserializerConfig {
1616
/// Vector's native JSON-specific decoding options.
17+
#[serde(
18+
default,
19+
skip_serializing_if = "vector_core::serde::skip_serializing_if_default"
20+
)]
1721
pub native_json: NativeJsonDeserializerOptions,
1822
}
1923

lib/codecs/src/decoding/format/syslog.rs

+8-2
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ use chrono::{DateTime, Datelike, Utc};
33
use derivative::Derivative;
44
use lookup::lookup_v2::parse_value_path;
55
use lookup::{event_path, owned_value_path, OwnedTargetPath, OwnedValuePath, PathPrefix};
6-
use serde::{Deserialize, Serialize};
76
use smallvec::{smallvec, SmallVec};
87
use std::borrow::Cow;
98
use std::collections::BTreeMap;
@@ -20,10 +19,17 @@ use vrl::value::{kind::Collection, Kind};
2019
use super::{default_lossy, Deserializer};
2120

2221
/// Config used to build a `SyslogDeserializer`.
23-
#[derive(Debug, Clone, Default, Deserialize, Serialize)]
22+
#[configurable_component]
23+
#[derive(Debug, Clone, Default)]
2424
pub struct SyslogDeserializerConfig {
25+
#[serde(skip)]
2526
source: Option<&'static str>,
27+
2628
/// Syslog-specific decoding options.
29+
#[serde(
30+
default,
31+
skip_serializing_if = "vector_core::serde::skip_serializing_if_default"
32+
)]
2733
pub syslog: SyslogDeserializerOptions,
2834
}
2935

lib/codecs/src/decoding/framing/character_delimited.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
use bytes::{Buf, Bytes, BytesMut};
22
use memchr::memchr;
3-
use serde::{Deserialize, Serialize};
43
use tokio_util::codec::Decoder;
54
use tracing::{trace, warn};
65
use vector_config::configurable_component;
76

87
use super::BoxedFramingError;
98

109
/// Config used to build a `CharacterDelimitedDecoder`.
11-
#[derive(Debug, Clone, Deserialize, Serialize)]
10+
#[configurable_component]
11+
#[derive(Debug, Clone)]
1212
pub struct CharacterDelimitedDecoderConfig {
1313
/// Options for the character delimited decoder.
1414
pub character_delimited: CharacterDelimitedDecoderOptions,

lib/codecs/src/decoding/framing/newline_delimited.rs

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,19 @@
11
use bytes::{Bytes, BytesMut};
22
use derivative::Derivative;
3-
use serde::{Deserialize, Serialize};
43
use tokio_util::codec::Decoder;
54
use vector_config::configurable_component;
65

76
use super::{BoxedFramingError, CharacterDelimitedDecoder};
87

98
/// Config used to build a `NewlineDelimitedDecoder`.
10-
#[derive(Debug, Clone, Default, Deserialize, Serialize, PartialEq, Eq)]
9+
#[configurable_component]
10+
#[derive(Debug, Clone, Default, PartialEq, Eq)]
1111
pub struct NewlineDelimitedDecoderConfig {
12+
/// Options for the newline delimited decoder.
1213
#[serde(
1314
default,
1415
skip_serializing_if = "vector_core::serde::skip_serializing_if_default"
1516
)]
16-
/// Options for the newline delimited decoder.
1717
pub newline_delimited: NewlineDelimitedDecoderOptions,
1818
}
1919

lib/codecs/src/decoding/framing/octet_counting.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,15 +2,15 @@ use std::io;
22

33
use bytes::{Buf, Bytes, BytesMut};
44
use derivative::Derivative;
5-
use serde::{Deserialize, Serialize};
65
use tokio_util::codec::{LinesCodec, LinesCodecError};
76
use tracing::trace;
87
use vector_config::configurable_component;
98

109
use super::BoxedFramingError;
1110

1211
/// Config used to build a `OctetCountingDecoder`.
13-
#[derive(Debug, Clone, Default, Deserialize, Serialize)]
12+
#[configurable_component]
13+
#[derive(Debug, Clone, Default)]
1414
pub struct OctetCountingDecoderConfig {
1515
#[serde(
1616
default,

0 commit comments

Comments
 (0)