EdColeman commented on code in PR #3887:
URL: https://github.com/apache/accumulo/pull/3887#discussion_r1371819466


##########
server/tserver/src/main/java/org/apache/accumulo/tserver/compaction/CompactionStrategy.java:
##########
@@ -28,11 +28,11 @@
 /**
  * The interface for customizing major compactions.
  * <p>
- * Important: Compaction configurations should use one compaction scheme, 
either the
- * {@link CompactionPlanner} introduced in 2.1, or a CompactionStrategy (see 
deprecation section).
+ * Important: Compaction configurations should use one compaction scheme, 
either the new

Review Comment:
   Updated text in 5c387050a2.  The intent of this is provide a strong warning 
to not use both compaction models at the same time and prompt developers to 
look elsewhere for the details - hopefully the new text conveys that.



-- 
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]

Reply via email to