weixiangsun commented on a change in pull request #8029:
URL: https://github.com/apache/pinot/pull/8029#discussion_r808724195
##########
File path:
pinot-core/src/main/java/org/apache/pinot/core/plan/CombinePlanNode.java
##########
@@ -161,8 +162,18 @@ public BaseCombineOperator run() {
// Streaming query (only support selection only)
return new StreamingSelectionOnlyCombineOperator(operators,
_queryContext, _executorService, _streamObserver);
}
+ GapfillUtils.GapfillType gapfillType =
GapfillUtils.getGapfillType(_queryContext);
Review comment:
The reason I need handle the gapfill query first because otherwise I
need make sure the query is not gapfill query for last two ifs. I prefer to
keep them as they are.
--
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
