ctubbsii commented on a change in pull request #2006:
URL: https://github.com/apache/accumulo/pull/2006#discussion_r613366737
##########
File path:
server/base/src/main/java/org/apache/accumulo/server/master/balancer/HostRegexTableLoadBalancer.java
##########
@@ -85,7 +85,7 @@
* <b>table.custom.balancer.host.regex.max.outstanding.migrations</b>
*
*/
-public class HostRegexTableLoadBalancer extends TableLoadBalancer implements
ConfigurationObserver {
Review comment:
Removing this interface could cause user code that assigns the variable
to a type, or any subclasses that explicitly override those methods to fail to
compile. I think it's probably low-risk enough that it's okay, but just wanted
to raise the point for consideration.
--
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]
