Skip to content

chore(codecs) update avro-rs to 0.22.0 - #26146

Merged
pront merged 9 commits into
vectordotdev:masterfrom
omwbennett:avro-rs-0.22.0
Aug 27, 2026
Merged

chore(codecs) update avro-rs to 0.22.0#26146
pront merged 9 commits into
vectordotdev:masterfrom
omwbennett:avro-rs-0.22.0

Conversation

@omwbennett

@omwbennett omwbennett commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Update to avro-rs 0.22.0 to enable us to use resolve_with_names() in #26112. Resolve a few breaking changes between 0.21.0 and 0.22.0 to get this working.

References

Related: #26112

Vector configuration

N/A

How did you test this PR?

Existing tests pass. Re-ran generate-avro-fixtures to update the syntax of Avro types in each fixture.

Is this a breaking change?

  • Yes
  • No (maybe?)

Upgrading to 0.22.0 fixes apache/avro-rs#476 (See apache/avro-rs#479) which means avro-rs will no longer parse invalid Avro schemas. I suppose this could be considered a breaking change if anyone is currently using an invalid Avro schema?

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on our guidelines.
  • No. A maintainer will apply the no-changelog label to this PR.

Contributor Guidelines

  • Please read our Vector contributor resources.
  • Do not hesitate to use @vectordotdev/vector to reach out to us regarding this PR.
  • Some CI checks run only after we manually approve them. To catch issues early, add a pre-push hook (template) or run the following locally before pushing:
    • make fmt
    • make check-clippy (auto-fix with make clippy-fix)
    • make test
  • After a review is requested, please avoid force pushes to help us review incrementally.
    • Feel free to push as many commits as you want. They will be squashed into one before merging.
    • For example, you can run git merge origin master and git push.
  • If this PR introduces changes Vector dependencies (modifies Cargo.lock), please
    run make build-licenses to regenerate the license inventory and commit the changes (if any). More details on the dd-rust-license-tool.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d03d7415f0

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

Comment thread Cargo.lock
Comment thread lib/codecs/tests/data/avro/generated/timestamp_millis.avsc Outdated
Comment thread lib/codecs/Cargo.toml
@pront pront added domain: codecs Anything related to Vector's codecs (encoding/decoding) meta: awaiting author Pull requests that are awaiting their author. labels Aug 25, 2026
@github-actions github-actions Bot removed domain: codecs Anything related to Vector's codecs (encoding/decoding) meta: awaiting author Pull requests that are awaiting their author. labels Aug 26, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0c36660ee7

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

Comment thread lib/codecs/src/encoding/format/avro.rs Outdated
Comment thread changelog.d/avro_strict_schema_parsing.breaking.md Outdated
@pront
pront enabled auto-merge August 27, 2026 14:57
auto-merge was automatically disabled August 27, 2026 17:30

Head branch was pushed to by a user without write access

@pront
pront enabled auto-merge August 27, 2026 17:33
@pront
pront added this pull request to the merge queue Aug 27, 2026
Merged via the queue into vectordotdev:master with commit 358c81e Aug 27, 2026
59 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

type.enum is valid while it should fail, good is type.type.enum according to spec

2 participants