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