keith-turner commented on a change in pull request #1605:
URL: https://github.com/apache/accumulo/pull/1605#discussion_r433510222
##########
File path:
core/src/main/java/org/apache/accumulo/core/client/admin/CompactionStrategyConfig.java
##########
@@ -30,27 +26,28 @@
* {@link CompactionConfig}.
*
* @since 1.7.0
+ * @deprecated since 2.1.0 CompactionStrategies were deprecated for multiple
reasons. First, they do
+ * not support the new compaction execution model. Second, they
bind selection and
+ * output file configuration into a single entity when users need
to configure these
+ * independently. Third, they use internal Accumulo types and
ensuring their stability
Review comment:
@ctubbsii did the answer and the slack chat we had resolve this or do
you think there is still something to be done?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
