-
Notifications
You must be signed in to change notification settings - Fork 311
HPCC-35430 Increase the max compression ratio to 50x #20690
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
Conversation
|
Jira Issue: https://hpccsystems.atlassian.net//browse/HPCC-35430 Jirabot Action Result: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR increases the maximum compression ratio for HPCC index tree leaf nodes from 25x to 50x, allowing more aggressive compression of highly compressible data while still limiting memory usage when the compressed data is read.
- Changed
maxCompressionFactorfrom 25 to 50 in theInplaceKeyBuildContextstructure - Updated the corresponding comment to reflect the new ratio (2% instead of 4%)
system/jhtree/jhinplace.hpp
Outdated
| struct { | ||
| double minCompressionThreshold = 0.95; // use uncompressed if compressed is > 95% uncompressed | ||
| unsigned maxCompressionFactor = 25; // Don't compress payload to less than 4% of the original by default (beause when it is read it will use lots of memory) | ||
| unsigned maxCompressionFactor = 50; // Don't compress payload to less than 2% of the original by default (beause when it is read it will use lots of memory) |
Copilot
AI
Nov 28, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling error: "beause" should be "because".
| unsigned maxCompressionFactor = 50; // Don't compress payload to less than 2% of the original by default (beause when it is read it will use lots of memory) | |
| unsigned maxCompressionFactor = 50; // Don't compress payload to less than 2% of the original by default (because when it is read it will use lots of memory) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, sure
mckellyln
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved.
But because is spelled incorrectly.
Signed-off-by: Gavin Halliday <[email protected]>
|
Jirabot Action Result: |
Type of change:
Checklist:
Smoketest:
Testing: