-
Notifications
You must be signed in to change notification settings - Fork 321
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.
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
zstd: Improve memory usage on small streaming encodes #1007
Conversation
Very small streams will use EncodeAll internally when closing and no header has been written. This will pull a new encoder from the async buffer. Instead re-use the stream encoder. Before: ``` BenchmarkMem/flush-32 1359 837989 ns/op 7376959 B/op 109 allocs/op BenchmarkMem/no-flush-32 129 8884753 ns/op 112044489 B/op 254 allocs/op ``` After: ``` BenchmarkMem/flush-32 1254 922593 ns/op 7376966 B/op 109 allocs/op BenchmarkMem/no-flush-32 1488 841270 ns/op 7374164 B/op 29 allocs/op ``` Test is pretty much worst case, but shows the issue nicely.
@coderabbitai review |
Actions performedReview triggered.
|
Caution Review failedThe pull request is closed. WalkthroughWalkthroughThe changes encompass updates to several Go modules within the Changes
Suggested reviewers
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
…5834) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/klauspost/compress](https://redirect.github.com/klauspost/compress) | `v1.17.9` -> `v1.17.10` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fklauspost%2fcompress/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fklauspost%2fcompress/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fklauspost%2fcompress/v1.17.9/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fklauspost%2fcompress/v1.17.9/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>klauspost/compress (github.com/klauspost/compress)</summary> ### [`v1.17.10`](https://redirect.github.com/klauspost/compress/releases/tag/v1.17.10) [Compare Source](https://redirect.github.com/klauspost/compress/compare/v1.17.9...v1.17.10) #### What's Changed - gzhttp: Add TransportAlwaysDecompress option. by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/978](https://redirect.github.com/klauspost/compress/pull/978) - s2: Add EncodeBuffer buffer recycling callback by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/982](https://redirect.github.com/klauspost/compress/pull/982) - zstd: Improve memory usage on small streaming encodes by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/1007](https://redirect.github.com/klauspost/compress/pull/1007) - gzhttp: Add supported decompress request body by [@​mirecl](https://redirect.github.com/mirecl) in [https://github.com/klauspost/compress/pull/1002](https://redirect.github.com/klauspost/compress/pull/1002) - flate: read data written with partial flush by [@​vajexal](https://redirect.github.com/vajexal) in [https://github.com/klauspost/compress/pull/996](https://redirect.github.com/klauspost/compress/pull/996) - ci: Upgrade Go & other by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/1008](https://redirect.github.com/klauspost/compress/pull/1008) - docs: Small typofix in comment by [@​Jille](https://redirect.github.com/Jille) in [https://github.com/klauspost/compress/pull/976](https://redirect.github.com/klauspost/compress/pull/976) - build(deps): bump the github-actions group with 2 updates by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/979](https://redirect.github.com/klauspost/compress/pull/979) - docs: Fix URL typo when installing builddict by [@​Wikidepia](https://redirect.github.com/Wikidepia) in [https://github.com/klauspost/compress/pull/980](https://redirect.github.com/klauspost/compress/pull/980) - build(deps): bump the github-actions group with 2 updates by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/985](https://redirect.github.com/klauspost/compress/pull/985) - Fix typos by [@​deining](https://redirect.github.com/deining) in [https://github.com/klauspost/compress/pull/986](https://redirect.github.com/klauspost/compress/pull/986) - build(deps): bump github/codeql-action from 3.25.15 to 3.26.6 in the github-actions group by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/997](https://redirect.github.com/klauspost/compress/pull/997) #### New Contributors - [@​Wikidepia](https://redirect.github.com/Wikidepia) made their first contribution in [https://github.com/klauspost/compress/pull/980](https://redirect.github.com/klauspost/compress/pull/980) - [@​deining](https://redirect.github.com/deining) made their first contribution in [https://github.com/klauspost/compress/pull/986](https://redirect.github.com/klauspost/compress/pull/986) - [@​vajexal](https://redirect.github.com/vajexal) made their first contribution in [https://github.com/klauspost/compress/pull/996](https://redirect.github.com/klauspost/compress/pull/996) - [@​mirecl](https://redirect.github.com/mirecl) made their first contribution in [https://github.com/klauspost/compress/pull/1002](https://redirect.github.com/klauspost/compress/pull/1002) **Full Changelog**: klauspost/compress@v1.17.9...v1.17.10 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/open-telemetry/opentelemetry-go). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC44MC4wIiwidXBkYXRlZEluVmVyIjoiMzguODAuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiU2tpcCBDaGFuZ2Vsb2ciLCJkZXBlbmRlbmNpZXMiXX0=--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…6146) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/klauspost/compress](https://redirect.github.com/klauspost/compress) | `v1.17.9` -> `v1.17.10` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fklauspost%2fcompress/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fklauspost%2fcompress/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fklauspost%2fcompress/v1.17.9/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fklauspost%2fcompress/v1.17.9/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>klauspost/compress (github.com/klauspost/compress)</summary> ### [`v1.17.10`](https://redirect.github.com/klauspost/compress/releases/tag/v1.17.10) [Compare Source](https://redirect.github.com/klauspost/compress/compare/v1.17.9...v1.17.10) #### What's Changed - gzhttp: Add TransportAlwaysDecompress option. by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/978](https://redirect.github.com/klauspost/compress/pull/978) - s2: Add EncodeBuffer buffer recycling callback by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/982](https://redirect.github.com/klauspost/compress/pull/982) - zstd: Improve memory usage on small streaming encodes by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/1007](https://redirect.github.com/klauspost/compress/pull/1007) - gzhttp: Add supported decompress request body by [@​mirecl](https://redirect.github.com/mirecl) in [https://github.com/klauspost/compress/pull/1002](https://redirect.github.com/klauspost/compress/pull/1002) - flate: read data written with partial flush by [@​vajexal](https://redirect.github.com/vajexal) in [https://github.com/klauspost/compress/pull/996](https://redirect.github.com/klauspost/compress/pull/996) - ci: Upgrade Go & other by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/1008](https://redirect.github.com/klauspost/compress/pull/1008) - docs: Small typofix in comment by [@​Jille](https://redirect.github.com/Jille) in [https://github.com/klauspost/compress/pull/976](https://redirect.github.com/klauspost/compress/pull/976) - build(deps): bump the github-actions group with 2 updates by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/979](https://redirect.github.com/klauspost/compress/pull/979) - docs: Fix URL typo when installing builddict by [@​Wikidepia](https://redirect.github.com/Wikidepia) in [https://github.com/klauspost/compress/pull/980](https://redirect.github.com/klauspost/compress/pull/980) - build(deps): bump the github-actions group with 2 updates by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/985](https://redirect.github.com/klauspost/compress/pull/985) - Fix typos by [@​deining](https://redirect.github.com/deining) in [https://github.com/klauspost/compress/pull/986](https://redirect.github.com/klauspost/compress/pull/986) - build(deps): bump github/codeql-action from 3.25.15 to 3.26.6 in the github-actions group by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/997](https://redirect.github.com/klauspost/compress/pull/997) #### New Contributors - [@​Wikidepia](https://redirect.github.com/Wikidepia) made their first contribution in [https://github.com/klauspost/compress/pull/980](https://redirect.github.com/klauspost/compress/pull/980) - [@​deining](https://redirect.github.com/deining) made their first contribution in [https://github.com/klauspost/compress/pull/986](https://redirect.github.com/klauspost/compress/pull/986) - [@​vajexal](https://redirect.github.com/vajexal) made their first contribution in [https://github.com/klauspost/compress/pull/996](https://redirect.github.com/klauspost/compress/pull/996) - [@​mirecl](https://redirect.github.com/mirecl) made their first contribution in [https://github.com/klauspost/compress/pull/1002](https://redirect.github.com/klauspost/compress/pull/1002) **Full Changelog**: klauspost/compress@v1.17.9...v1.17.10 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/open-telemetry/opentelemetry-go-contrib). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC44MC4wIiwidXBkYXRlZEluVmVyIjoiMzguODAuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiU2tpcCBDaGFuZ2Vsb2ciLCJkZXBlbmRlbmNpZXMiXX0=--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/klauspost/compress](https://redirect.github.com/klauspost/compress) | `v1.17.9` -> `v1.17.10` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fklauspost%2fcompress/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fklauspost%2fcompress/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fklauspost%2fcompress/v1.17.9/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fklauspost%2fcompress/v1.17.9/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>klauspost/compress (github.com/klauspost/compress)</summary> ### [`v1.17.10`](https://redirect.github.com/klauspost/compress/releases/tag/v1.17.10) [Compare Source](https://redirect.github.com/klauspost/compress/compare/v1.17.9...v1.17.10) #### What's Changed - gzhttp: Add TransportAlwaysDecompress option. by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/978](https://redirect.github.com/klauspost/compress/pull/978) - s2: Add EncodeBuffer buffer recycling callback by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/982](https://redirect.github.com/klauspost/compress/pull/982) - zstd: Improve memory usage on small streaming encodes by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/1007](https://redirect.github.com/klauspost/compress/pull/1007) - gzhttp: Add supported decompress request body by [@​mirecl](https://redirect.github.com/mirecl) in [https://github.com/klauspost/compress/pull/1002](https://redirect.github.com/klauspost/compress/pull/1002) - flate: read data written with partial flush by [@​vajexal](https://redirect.github.com/vajexal) in [https://github.com/klauspost/compress/pull/996](https://redirect.github.com/klauspost/compress/pull/996) - ci: Upgrade Go & other by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/1008](https://redirect.github.com/klauspost/compress/pull/1008) - docs: Small typofix in comment by [@​Jille](https://redirect.github.com/Jille) in [https://github.com/klauspost/compress/pull/976](https://redirect.github.com/klauspost/compress/pull/976) - build(deps): bump the github-actions group with 2 updates by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/979](https://redirect.github.com/klauspost/compress/pull/979) - docs: Fix URL typo when installing builddict by [@​Wikidepia](https://redirect.github.com/Wikidepia) in [https://github.com/klauspost/compress/pull/980](https://redirect.github.com/klauspost/compress/pull/980) - build(deps): bump the github-actions group with 2 updates by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/985](https://redirect.github.com/klauspost/compress/pull/985) - Fix typos by [@​deining](https://redirect.github.com/deining) in [https://github.com/klauspost/compress/pull/986](https://redirect.github.com/klauspost/compress/pull/986) - build(deps): bump github/codeql-action from 3.25.15 to 3.26.6 in the github-actions group by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/997](https://redirect.github.com/klauspost/compress/pull/997) #### New Contributors - [@​Wikidepia](https://redirect.github.com/Wikidepia) made their first contribution in [https://github.com/klauspost/compress/pull/980](https://redirect.github.com/klauspost/compress/pull/980) - [@​deining](https://redirect.github.com/deining) made their first contribution in [https://github.com/klauspost/compress/pull/986](https://redirect.github.com/klauspost/compress/pull/986) - [@​vajexal](https://redirect.github.com/vajexal) made their first contribution in [https://github.com/klauspost/compress/pull/996](https://redirect.github.com/klauspost/compress/pull/996) - [@​mirecl](https://redirect.github.com/mirecl) made their first contribution in [https://github.com/klauspost/compress/pull/1002](https://redirect.github.com/klauspost/compress/pull/1002) **Full Changelog**: klauspost/compress@v1.17.9...v1.17.10 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/XSAM/otelsql). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC44MC4wIiwidXBkYXRlZEluVmVyIjoiMzguODAuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…11256) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/klauspost/compress](https://redirect.github.com/klauspost/compress) | `v1.17.9` -> `v1.17.10` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fklauspost%2fcompress/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fklauspost%2fcompress/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fklauspost%2fcompress/v1.17.9/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fklauspost%2fcompress/v1.17.9/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>klauspost/compress (github.com/klauspost/compress)</summary> ### [`v1.17.10`](https://redirect.github.com/klauspost/compress/releases/tag/v1.17.10) [Compare Source](https://redirect.github.com/klauspost/compress/compare/v1.17.9...v1.17.10) #### What's Changed - gzhttp: Add TransportAlwaysDecompress option. by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/978](https://redirect.github.com/klauspost/compress/pull/978) - s2: Add EncodeBuffer buffer recycling callback by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/982](https://redirect.github.com/klauspost/compress/pull/982) - zstd: Improve memory usage on small streaming encodes by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/1007](https://redirect.github.com/klauspost/compress/pull/1007) - gzhttp: Add supported decompress request body by [@​mirecl](https://redirect.github.com/mirecl) in [https://github.com/klauspost/compress/pull/1002](https://redirect.github.com/klauspost/compress/pull/1002) - flate: read data written with partial flush by [@​vajexal](https://redirect.github.com/vajexal) in [https://github.com/klauspost/compress/pull/996](https://redirect.github.com/klauspost/compress/pull/996) - ci: Upgrade Go & other by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/1008](https://redirect.github.com/klauspost/compress/pull/1008) - docs: Small typofix in comment by [@​Jille](https://redirect.github.com/Jille) in [https://github.com/klauspost/compress/pull/976](https://redirect.github.com/klauspost/compress/pull/976) - build(deps): bump the github-actions group with 2 updates by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/979](https://redirect.github.com/klauspost/compress/pull/979) - docs: Fix URL typo when installing builddict by [@​Wikidepia](https://redirect.github.com/Wikidepia) in [https://github.com/klauspost/compress/pull/980](https://redirect.github.com/klauspost/compress/pull/980) - build(deps): bump the github-actions group with 2 updates by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/985](https://redirect.github.com/klauspost/compress/pull/985) - Fix typos by [@​deining](https://redirect.github.com/deining) in [https://github.com/klauspost/compress/pull/986](https://redirect.github.com/klauspost/compress/pull/986) - build(deps): bump github/codeql-action from 3.25.15 to 3.26.6 in the github-actions group by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/997](https://redirect.github.com/klauspost/compress/pull/997) #### New Contributors - [@​Wikidepia](https://redirect.github.com/Wikidepia) made their first contribution in [https://github.com/klauspost/compress/pull/980](https://redirect.github.com/klauspost/compress/pull/980) - [@​deining](https://redirect.github.com/deining) made their first contribution in [https://github.com/klauspost/compress/pull/986](https://redirect.github.com/klauspost/compress/pull/986) - [@​vajexal](https://redirect.github.com/vajexal) made their first contribution in [https://github.com/klauspost/compress/pull/996](https://redirect.github.com/klauspost/compress/pull/996) - [@​mirecl](https://redirect.github.com/mirecl) made their first contribution in [https://github.com/klauspost/compress/pull/1002](https://redirect.github.com/klauspost/compress/pull/1002) **Full Changelog**: klauspost/compress@v1.17.9...v1.17.10 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/open-telemetry/opentelemetry-collector). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC44MC4wIiwidXBkYXRlZEluVmVyIjoiMzguODAuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiZGVwZW5kZW5jaWVzIiwicmVub3ZhdGVib3QiXX0=--> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
…35385) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/klauspost/compress](https://redirect.github.com/klauspost/compress) | `v1.17.9` -> `v1.17.10` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fklauspost%2fcompress/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fklauspost%2fcompress/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fklauspost%2fcompress/v1.17.9/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fklauspost%2fcompress/v1.17.9/v1.17.10?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>klauspost/compress (github.com/klauspost/compress)</summary> ### [`v1.17.10`](https://redirect.github.com/klauspost/compress/releases/tag/v1.17.10) [Compare Source](https://redirect.github.com/klauspost/compress/compare/v1.17.9...v1.17.10) #### What's Changed - gzhttp: Add TransportAlwaysDecompress option. by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/978](https://redirect.github.com/klauspost/compress/pull/978) - s2: Add EncodeBuffer buffer recycling callback by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/982](https://redirect.github.com/klauspost/compress/pull/982) - zstd: Improve memory usage on small streaming encodes by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/1007](https://redirect.github.com/klauspost/compress/pull/1007) - gzhttp: Add supported decompress request body by [@​mirecl](https://redirect.github.com/mirecl) in [https://github.com/klauspost/compress/pull/1002](https://redirect.github.com/klauspost/compress/pull/1002) - flate: read data written with partial flush by [@​vajexal](https://redirect.github.com/vajexal) in [https://github.com/klauspost/compress/pull/996](https://redirect.github.com/klauspost/compress/pull/996) - ci: Upgrade Go & other by [@​klauspost](https://redirect.github.com/klauspost) in [https://github.com/klauspost/compress/pull/1008](https://redirect.github.com/klauspost/compress/pull/1008) - docs: Small typofix in comment by [@​Jille](https://redirect.github.com/Jille) in [https://github.com/klauspost/compress/pull/976](https://redirect.github.com/klauspost/compress/pull/976) - build(deps): bump the github-actions group with 2 updates by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/979](https://redirect.github.com/klauspost/compress/pull/979) - docs: Fix URL typo when installing builddict by [@​Wikidepia](https://redirect.github.com/Wikidepia) in [https://github.com/klauspost/compress/pull/980](https://redirect.github.com/klauspost/compress/pull/980) - build(deps): bump the github-actions group with 2 updates by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/985](https://redirect.github.com/klauspost/compress/pull/985) - Fix typos by [@​deining](https://redirect.github.com/deining) in [https://github.com/klauspost/compress/pull/986](https://redirect.github.com/klauspost/compress/pull/986) - build(deps): bump github/codeql-action from 3.25.15 to 3.26.6 in the github-actions group by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/klauspost/compress/pull/997](https://redirect.github.com/klauspost/compress/pull/997) #### New Contributors - [@​Wikidepia](https://redirect.github.com/Wikidepia) made their first contribution in [https://github.com/klauspost/compress/pull/980](https://redirect.github.com/klauspost/compress/pull/980) - [@​deining](https://redirect.github.com/deining) made their first contribution in [https://github.com/klauspost/compress/pull/986](https://redirect.github.com/klauspost/compress/pull/986) - [@​vajexal](https://redirect.github.com/vajexal) made their first contribution in [https://github.com/klauspost/compress/pull/996](https://redirect.github.com/klauspost/compress/pull/996) - [@​mirecl](https://redirect.github.com/mirecl) made their first contribution in [https://github.com/klauspost/compress/pull/1002](https://redirect.github.com/klauspost/compress/pull/1002) **Full Changelog**: klauspost/compress@v1.17.9...v1.17.10 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC44MC4wIiwidXBkYXRlZEluVmVyIjoiMzguODAuMCIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiZGVwZW5kZW5jaWVzIiwicmVub3ZhdGVib3QiXX0=--> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
Very small streams will use EncodeAll internally when closing and no header has been written.
This will pull a new encoder from the async buffer.
Instead re-use the stream encoder.
Before:
After:
Test is pretty much worst case, but shows the issue nicely.
Summary by CodeRabbit
New Features
Bug Fixes
Refactor
Tests