Jackie-Jiang commented on code in PR #9282:
URL: https://github.com/apache/pinot/pull/9282#discussion_r957827047
##########
pinot-core/src/main/java/org/apache/pinot/core/operator/InstanceResponseOperator.java:
##########
@@ -34,24 +34,21 @@
public class InstanceResponseOperator extends
BaseOperator<InstanceResponseBlock> {
-
private static final String EXPLAIN_NAME = "INSTANCE_RESPONSE";
- private final BaseCombineOperator _combineOperator;
+ private final BaseCombineOperator<?> _combineOperator;
Review Comment:
We only need that in the combine operator because we need to know the `T`
when merging the blocks. the `getDataTable()` is a common interface, so we
don't need to know the exact type
--
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
