milleruntime commented on code in PR #2708:
URL: https://github.com/apache/accumulo/pull/2708#discussion_r873778785
##########
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##########
@@ -683,10 +683,10 @@ public enum Property {
"After a tablet has been idle (no mutations) for this time period it may
have its "
+ "in-memory map flushed to disk in a minor compaction. There is no
guarantee an idle "
+ "tablet will be compacted."),
- TABLE_MINC_MAX_MERGE_FILE_SIZE("table.compaction.minor.merge.file.size.max",
"0",
+ TABLE_MINC_MAX_MERGE_FILE_SIZE("table.compaction.minor.merge.file.size.max",
"1",
PropertyType.MEMORY,
- "The max file size used for a merging minor compaction. The default
value"
- + " of 0 disables a max file size."),
+ "The max file size used for a merging minor compaction. The value of 0
is no max "
+ + "file size. The default value is 1 byte, which should disable
merging minor compactions."),
Review Comment:
The only thing I could find was @keith-turner 's explanation in the release
notes for 2.1:
"Merging minor compactions could cause O(N^2) compaction work. "
https://accumulo.apache.org/release/accumulo-2.1.0/
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
