keith-turner commented on code in PR #4127:
URL: https://github.com/apache/accumulo/pull/4127#discussion_r1442380412
##########
core/src/main/java/org/apache/accumulo/core/spi/compaction/DefaultCompactionPlanner.java:
##########
@@ -108,6 +109,16 @@
* number of files that will be included in a single compaction.
* </ul>
*
+ * <p>
+ * Starting with Accumulo 2.1.3, this plugin will use the table config option
+ * {@code "table.file.max"}. When a tablet has no compactions running, its
number of files exceeds
+ * table.file.max, and system compactions are not finding anything to compact
then this plugin will
+ * try to find a lower compaction ratio that will result in a compaction. For
example if the
+ * compaction ratio is set to 3, a tablet has 20 files, table.file.max is 15
and no compactions are
+ * planned, then this plugin will find the largest compaction ratio less than
3 that results in a
+ * compaction.
+ * </p>
+ *
Review Comment:
Thats easier to understand. I updated the javadoc, made it use HTML for the
list and added a 4th condition that the code was checking for.
--
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]
