yashmayya commented on code in PR #15930:
URL: https://github.com/apache/pinot/pull/15930#discussion_r2122830984


##########
pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/TableRebalanceIntegrationTest.java:
##########
@@ -88,6 +88,78 @@ private String getRebalanceUrl(RebalanceConfig 
rebalanceConfig, TableType tableT
         + "?type=" + tableType.toString() + "&" + 
getQueryString(rebalanceConfig);
   }
 
+  @Test
+  public void testImplicitRealtimeTableInstanceAssignment() throws Exception {

Review Comment:
   Ah, good suggestion, I had avoided doing so assuming that it would be too 
cumbersome to setup or mock Kafka or an equivalent source stream in order to do 
some repartitioning. But the `FakeStream` works perfectly for this use case.



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