somandal commented on code in PR #16135:
URL: https://github.com/apache/pinot/pull/16135#discussion_r2153311201


##########
pinot-controller/src/test/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancerClusterStatelessTest.java:
##########
@@ -1331,10 +1303,8 @@ public void testRebalancePreCheckerRebalanceConfig()
   }
 
   /**
-   * Tests rebalance with tier configs
-   * Add 10 segments, with segment metadata end time 3 days apart starting 
from now to 30 days ago
-   * 1. run rebalance - should see no change
-   * 2. add nodes for tiers and run rebalance - should see no change
+   * Tests rebalance with tier configs Add 10 segments, with segment metadata 
end time 3 days apart starting from now to
+   * 30 days ago 1. run rebalance - should see no change 2. add nodes for 
tiers and run rebalance - should see no change

Review Comment:
   nit: the earlier formatting was somewhat easier to read, is this change 
intentional?



-- 
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: commits-unsubscr...@pinot.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to