Skip to content

Conversation

@JakubOnderka
Copy link
Contributor

Summary

zlib-rs is much faster for library for zlib decoding/encoding than default miniz_oxide and it is even faster than zlib-ng (https://trifectatech.org/blog/zlib-rs-is-faster-than-c/)

Change Type

  • Bug fix
  • New feature
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

How did you test this PR?

Standard tests.

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.

Checklist

  • Our CONTRIBUTING.md is a good starting place.
  • If this PR introduces changes to LICENSE-3rdparty.csv, please
    run dd-rust-license-tool write and commit the changes. More details here.
  • For new VRL functions, please also create a sibling PR in Vector to document the new function.

@JakubOnderka
Copy link
Contributor Author

Tests should pass now.

@pront
Copy link
Member

pront commented Mar 28, 2025

Hi @JakubOnderka, this is great. Can you add a changelog fragment please? It can be short, I just want to keep track of this in case something breaks after the release.

@pront pront self-requested a review March 28, 2025 14:17
@JakubOnderka
Copy link
Contributor Author

Changelog added.

@pront pront enabled auto-merge March 31, 2025 14:55
@pront
Copy link
Member

pront commented Mar 31, 2025

🚀

@pront pront added this pull request to the merge queue Mar 31, 2025
Merged via the queue into vectordotdev:main with commit d2ab928 Mar 31, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants