8000 performance(stdlib): Use zlib-rs for much faster zlib decoding/encoding by JakubOnderka · Pull Request #1301 · vectordotdev/vrl · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

performance(stdlib): Use zlib-rs for much faster zlib decoding/encoding #1301

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Mar 31, 2025
Merged

performance(stdlib): Use zlib-rs for much faster zlib decoding/encoding #1301

merged 3 commits into from
Mar 31, 2025

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 JakubOnderka mentioned this pull request Feb 27, 2025
Merged
10 tasks
@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
@pront pront mentioned this pull request Apr 1, 2025
Merged
@JakubOnderka JakubOnderka mentioned this pull request Apr 2, 2025
Merged
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pront pront pront approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JakubOnderka @pront
0