| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
There was a problem hiding this comment.
Here are some automated review suggestions for this pull request.
Reviewed commit: 0ca77c6bfb
ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Sorry, something went wrong.
There was a problem hiding this comment.
Here are some automated review suggestions for this pull request.
Reviewed commit: 4808acf3b0
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Sorry, something went wrong.
Head branch was pushed to by a user without write access
There was a problem hiding this comment.
Here are some automated review suggestions for this pull request.
Reviewed commit: 9b6ef3c523
ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Sorry, something went wrong.
There was a problem hiding this comment.
Here are some automated review suggestions for this pull request.
Reviewed commit: 5a43236ddf
ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Summary
Support encoding and decoding Avro Date and TimeMillis values.
Due to the way VRL stores integers, an extra step was added to the avro encoder path to coerce Avro types based on the schema field's logical type.
References
Closes #24773
Vector configuration
sources: kafka_avro: type: kafka bootstrap_servers: "kafka:9092" group_id: "vector" topics: - test_topic decoding: codec: avro avro: schema: | { "type": "record", "name": "test", "fields": [ { "name": "date_field", "type": { "type": "int", "logicalType": "date" } } ] } strip_schema_id_prefix: false sinks: console_out: type: console inputs: - kafka_avro encoding: codec: jsonAnd similar for TimeMillis.
How did you test this PR?
Manual testing with the above configs, added unit tests for coerce_logical_types and enabled the existing date/timemillis avro round trip tests. All tests passing.
Is this a breaking change?
Does this PR include user facing changes?
Contributor Guidelines
run make build-licenses to regenerate the license inventory and commit the changes (if any). More details on the dd-rust-license-tool.