native_json
codec is not serializing/deserializing EventMetadata
.
#18570
Labels
domain: codecs
Anything related to Vector's codecs (encoding/decoding)
domain: tests
Anything related to Vector's internal tests
meta: regression
This issue represents a regression
type: bug
A code related bug.
Discovered while adding to the Vector protobuf definition for the native codec, as part of #18405.
Doing so necessitated regenerating the fixture files used in the native codec unit tests.
This led to a test failure in
current_native_decoding_matches
, which compares the Event that is created by deserializng a json fixture file with a proto fixture file, that should have been encoded with the same Event when the files were generated.The offending bits are that the
EventMetadata.value
is not being handled by thenative_json
codec, but it is being handled by thenative
(protobuf) codec. This boils down to the fact that the EventMetadata is being skipped by the serde deserialize/serialize for the Event structs. This could largely be due to the fact that not skipping that field means that theEventMetadata
must satisfy the trait requirements ofconfigurable_component
.There are basically three problems here:
native_json
codec it seems should be handling thevalue
field ofEventMetadata
.current_native_decoding_matches
fails on master due to 1The text was updated successfully, but these errors were encountered: