jasperjiaguo commented on code in PR #9236:
URL: https://github.com/apache/pinot/pull/9236#discussion_r954297069
##########
pinot-core/src/test/java/org/apache/pinot/queries/BaseQueriesTest.java:
##########
@@ -91,7 +95,9 @@ protected <T extends Operator> T getOperatorWithFilter(String
query) {
/**
* Run query on multiple index segments.
* <p>Use this to test the whole flow from server to broker.
- * <p>The result should be equivalent to querying 4 identical index segments.
+ * <p>Unless explicitly override getDistinctInstances or initialize 2
distinct index segments in test, the result
Review Comment:
From your code it seems overriding `getDistinctInstances` and initializing 2
instances is the way to trigger the new `getBrokerResponseDistinctInstances `
behavior? Also the user should take care of assigning the distinct segments
correctly to the two instances. Maybe update this comment a little bit and also
add a pointer to the usage `CovarianceQueriesTest`?
--
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
