Skip to content

Conversation

@surban
Copy link
Contributor

@surban surban commented Jul 11, 2025

Adds XzEncoder::parallel and XzDecoder::parallel methods that enable multi-threaded compression and decompression.

Care must be taken not that call flush() on the encoder in quick succession, as this would severely impact multi-threaded compression performance.

Gated by the xz-parallel crate feature.

Copy link
Collaborator

@NobodyXu NobodyXu left a comment

Choose a reason for hiding this comment

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

Thanks, just some minor change to API

@surban surban requested a review from NobodyXu July 12, 2025 15:55
Copy link
Collaborator

@NobodyXu NobodyXu left a comment

Choose a reason for hiding this comment

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

Thanks LGTM, @robjtede can you have a double check please?

Copy link
Member

@robjtede robjtede left a comment

Choose a reason for hiding this comment

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

cool 👍🏻

@robjtede robjtede added this pull request to the merge queue Jul 13, 2025
Merged via the queue into Nullus157:main with commit 3b884ed Jul 13, 2025
17 checks passed
@codecov
Copy link

codecov bot commented Jul 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (8f9394d) to head (a75df3c).
Report is 5 commits behind head on main.

Additional details and impacted files
@@     Coverage Diff     @@
##   main   #353   +/-   ##
===========================
===========================

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions github-actions bot mentioned this pull request Jul 13, 2025
@robjtede robjtede added the A-semver-minor non-breaking change label Jul 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-semver-minor non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants