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
Add static node settings to set default values for max merged segment sizes #102208
Add static node settings to set default values for max merged segment sizes #102208
Conversation
Hi @tlrx, I've created a changelog YAML for you. |
Pinging @elastic/es-distributed (Team:Distributed) |
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.
LGTM.
|
||
public static final double DEFAULT_EXPUNGE_DELETES_ALLOWED = 10d; | ||
public static final ByteSizeValue DEFAULT_FLOOR_SEGMENT = new ByteSizeValue(2, ByteSizeUnit.MB); | ||
public static final int DEFAULT_MAX_MERGE_AT_ONCE = 10; | ||
public static final ByteSizeValue DEFAULT_MAX_MERGED_SEGMENT = new ByteSizeValue(5, ByteSizeUnit.GB); | ||
public static final Setting<ByteSizeValue> DEFAULT_MAX_MERGED_SEGMENT_SETTING = Setting.byteSizeSetting( | ||
"merge.policy.default.max_merged_segment", |
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.
I'd prefer something like:
"merge.policy.default.max_merged_segment", | |
"indices.merge.policy.max_merged_segment", |
though adding default
could also be fine by me: indices.default.merge.policy.max_merged_segment
.
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.
I pushed 63a93ec to use indices.merge.policy.max_merged_segment
Thanks Henning! |
… sizes (elastic#102208) This change introduces 2 static node-level settings that allow to override the default values for the maximum merged segment sizes in tiered and time-based merge policies. New default values can be defined like this in elasticsearch.yml: indices: merge: policy: max_merged_segment: "5gb" max_time_based_merged_segment: "25gb"
This change introduces 2 static node-level settings that allow to override the default values for the maximum merged segment sizes in tiered and time-based merge policies.
New default values can be defined like this in
elasticsearch.yml
: