Skip to content

Conversation

@austinderek
Copy link
Owner

avoid duplicate code in compression dictionary creation

avoid multiple iterations for compression dictionary

fixup

fixup

fixup compute_dictionary

update zstd


🔄 This is a mirror of upstream PR #82336

@staging
Copy link

staging bot commented Aug 4, 2025

✅ This pull request was succesfully scanned and it was determined that it does not contain any changes that present immediate security concerns. If you would still like for it to be reviewed by an expert from our reviewer community, you can submit it manually via the HackerOne PullRequest dashboard.

⏱️ Latest scan covered changes up to commit aef6cf5 (latest)

@austinderek austinderek force-pushed the canary branch 26 times, most recently from 2da86a4 to 4e06942 Compare August 5, 2025 15:32
@austinderek austinderek force-pushed the sokra/compression-dict-cleanup branch from aef6cf5 to 86b9cba Compare August 5, 2025 15:32
@staging
Copy link

staging bot commented Aug 5, 2025

🔍 This pull request has been sent to HackerOne's PullRequest review team because our automation detected one or more changes with potential security impact or requires further evaluation. Experts are now being assigned to this review based on relevant expertise and will validate or dismiss any security findings accordingly and post their feedback as comments within this pull request.

⏱️ Latest scan covered changes up to commit 86b9cba (latest)


Check the status or cancel this secure code review here.

@austinderek austinderek force-pushed the canary branch 27 times, most recently from 24d7fcc to 63cdc65 Compare August 10, 2025 04:06
@staging
Copy link

staging bot commented Aug 12, 2025

PullRequest network review has been cancelled

You can reactivate the code review job from the PullRequest dashboard - or - by adding [pr] to the title of this code review.

@austinderek austinderek deleted the sokra/compression-dict-cleanup branch August 13, 2025 09:10
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.

3 participants