ctubbsii commented on code in PR #5358:
URL: https://github.com/apache/accumulo/pull/5358#discussion_r1971937750


##########
core/src/main/java/org/apache/accumulo/core/spi/balancer/BalancerEnvironment.java:
##########
@@ -40,6 +40,12 @@ public interface BalancerEnvironment extends 
ServiceEnvironment {
    * Many Accumulo plugins are given table IDs as this is what Accumulo uses 
internally to identify
    * tables. This provides a mapping of table names to table IDs for the 
purposes of translating
    * and/or enumerating the existing tables.
+   *
+   * <P>
+   * This returns all tables that exists in the system. Each request to 
balance should limit itself
+   * to {@link TabletBalancer.BalanceParameters#getTablesToBalance()} and not 
balance everything
+   * returned by this.
+   * </P>

Review Comment:
   ```suggestion
      * <p>
      * This returns all tables that exists in the system. Each request to 
balance should limit itself
      * to {@link TabletBalancer.BalanceParameters#getTablesToBalance()} and 
not balance everything
      * returned by 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.

To unsubscribe, e-mail: notifications-unsubscr...@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to