ccoffline commented on issue #4772:
URL: 
https://github.com/apache/incubator-doris/issues/4772#issuecomment-716976794


   > I forget to submit a PR to the Github master branch.
   > You could change the code to, and submit a PR.
   > 
   > ```
   >         if (analyzer.hasEmptyResultSet()) {
   >             List<TupleId> scanTupleIds = root.getAllScanTupleIds();
   >             PlanNode node = createEmptyNode(stmt, analyzer);
   >             // Ensure result exprs will be substituted by right outputSmap
   >             node.setOutputSmap(root.outputSmap);
   > 
   >             // Must and could only remove the scanNode that the tuple id 
is same with EmptyNode,
   >             // otherwise we will get NPE in 
Coordinator::computeScanRangeAssignment
   >             scanNodes.removeIf(scanNode -> 
scanTupleIds.contains(scanNode.getTupleIds().get(0)));
   >             // Currently, getMaterializedTupleIds for AnalyticEvalNode is 
wrong,
   >             // So we explicitly add AnalyticEvalNode tuple ids to 
EmptySetNode
   >             if (root instanceof AnalyticEvalNode) {
   >                 node.getTupleIds().addAll(root.tupleIds);
   >             }
   >             return node;
   >         }
   > ```
   
   I can't find method `getAllScanTupleIds` in `PlanNode root`


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to