Skip to content

Rust: Use zstd compression for TRAP files by default - #22375

Merged
hvitved merged 1 commit into
github:mainfrom
hvitved:rust/trap-compression-zstd-default
Aug 19, 2026
Merged

Rust: Use zstd compression for TRAP files by default#22375
hvitved merged 1 commit into
github:mainfrom
hvitved:rust/trap-compression-zstd-default

Conversation

@hvitved

@hvitvedhvitved commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

In a local experiment this reduces the trap folder from 1.69 GB to 293 MB.

@github-actionsgithub-actionsBot added the Rust Pull requests that update Rust code label Aug 19, 2026
@hvitvedhvitved added the no-change-note-required This PR does not need a change note label Aug 19, 2026
@hvitved
hvitved marked this pull request as ready for review August 19, 2026 08:09
@hvitved
hvitved requested review from a team as code ownersAugust 19, 2026 08:09
CopilotAI balanced review requested due to automatic review settings August 19, 2026 08:09

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Changes Rust extraction to use Zstandard compression by default, reducing TRAP disk usage.

Changes:

  • Makes Zstd the default TRAP compression mode.
Show a summary per file
FileDescription
rust/extractor/src/config.rsSets Zstandard as the default compression format.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment threadrust/extractor/src/config.rs
redsun82
redsun82 previously approved these changes Aug 19, 2026
@hvitved
hvitvedforce-pushed the rust/trap-compression-zstd-default branch from 9c6d3d4 to 29f91ccCompareAugust 19, 2026 08:23
@hvitved
hvitved requested a review from redsun82August 19, 2026 08:24
@hvitved
hvitved merged commit 7f60101 into github:mainAug 19, 2026
14 checks passed
@hvitved
hvitved deleted the rust/trap-compression-zstd-default branch August 19, 2026 09:44
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-change-note-requiredThis PR does not need a change noteRustPull requests that update Rust code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@hvitved@redsun82