brianloss commented on a change in pull request #1891:
URL: https://github.com/apache/accumulo/pull/1891#discussion_r567483942
##########
File path:
server/base/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java
##########
@@ -56,7 +56,11 @@
* Implementations may wish to store configuration in Accumulo's system
configuration using the
* {@link Property#GENERAL_ARBITRARY_PROP_PREFIX}. They may also benefit from
using per-table
* configuration using {@link Property#TABLE_ARBITRARY_PROP_PREFIX}.
+ *
+ * @deprecated since 2.1.0. Use {@link
org.apache.accumulo.core.spi.balancer.TabletBalancer}
+ * instead.
*/
+@Deprecated(since = "2.1.0")
public abstract class TabletBalancer {
Review comment:
> @brianloss I was thinking that AssignmentParamsImpl could be created
w/ refs to the new and old types and have methods to get the old types. The
could avoid the conversion from new to old. I am not sure but it seems like the
method that creates AssignmentParamsImpl has refs to the old types.
@keith-turner I pushed some changes that I think take care of this.
----------------------------------------------------------------
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]
