Skip to content
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

Ignore frozen chunks in compression policy #7837

Merged
merged 1 commit into from
Mar 17, 2025
Merged

Conversation

antekresic
Copy link
Contributor

Freezing a partially compressed chunk prevents us from compressing it back to fully compressed. If the chunk was frozen, we can just ignore them and continue with the rest of the chunks.

@antekresic antekresic added the bug label Mar 14, 2025
@antekresic antekresic added this to the v2.19.0 milestone Mar 14, 2025
@antekresic antekresic self-assigned this Mar 14, 2025
@antekresic antekresic force-pushed the antekresic/fix_policy branch 2 times, most recently from c709cc3 to 2e5f353 Compare March 16, 2025 10:36
@antekresic antekresic marked this pull request as ready for review March 16, 2025 10:36
@antekresic antekresic requested a review from svenklemm March 16, 2025 10:36
@antekresic antekresic added the force-auto-backport Automatically backport this PR or fix of this issue, even if it's not marked as "bug" label Mar 16, 2025
@antekresic antekresic force-pushed the antekresic/fix_policy branch 7 times, most recently from de6c1a7 to 256307e Compare March 17, 2025 09:56
@antekresic antekresic force-pushed the antekresic/fix_policy branch from 256307e to 21e0af6 Compare March 17, 2025 10:46
Freezing a partially compressed chunk prevents us from compressing
it back to fully compressed. If the chunk was frozen, we can just
ignore them and continue with the rest of the chunks.
@antekresic antekresic force-pushed the antekresic/fix_policy branch from 21e0af6 to b05272c Compare March 17, 2025 10:47
@antekresic antekresic merged commit e3cabc8 into main Mar 17, 2025
46 checks passed
@antekresic antekresic deleted the antekresic/fix_policy branch March 17, 2025 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported-2.19.x bug force-auto-backport Automatically backport this PR or fix of this issue, even if it's not marked as "bug"
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants