Jackie-Jiang commented on code in PR #9282:
URL: https://github.com/apache/pinot/pull/9282#discussion_r957812873
##########
pinot-core/src/main/java/org/apache/pinot/core/operator/AcquireReleaseColumnsSegmentOperator.java:
##########
@@ -58,8 +58,8 @@ public AcquireReleaseColumnsSegmentOperator(PlanNode
planNode, IndexSegment inde
* Runs the planNode to get the childOperator, and then proceeds with
execution.
*/
@Override
- protected IntermediateResultsBlock getNextBlock() {
- _childOperator = (Operator<IntermediateResultsBlock>) _planNode.run();
+ protected BaseResultsBlock getNextBlock() {
+ _childOperator = (Operator<BaseResultsBlock>) _planNode.run();
return _childOperator.nextBlock();
Review Comment:
This is a workaround to acquire the segment before running the plan.
Currently the plan is physical execution plan, which is per-segment based.
--
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
