jadami10 commented on code in PR #11768:
URL: https://github.com/apache/pinot/pull/11768#discussion_r1350970742


##########
pinot-controller/src/test/java/org/apache/pinot/controller/helix/core/rebalance/TableRebalancerClusterStatelessTest.java:
##########
@@ -66,13 +74,16 @@ public class TableRebalancerClusterStatelessTest extends 
ControllerTest {
   private static final String TIER_A_NAME = "tierA";
   private static final String TIER_B_NAME = "tierB";
   private static final String TIER_FIXED_NAME = "tierFixed";
+  private static final TableRebalanceObserver TABLE_REBALANCE_OBSERVER = 
mock(TableRebalanceObserver.class);

Review Comment:
   this class was already testing a bunch of cases, so adding a mock here to 
check the observer seemed the best way to test.



-- 
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