korlov42 commented on a change in pull request #9084:
URL: https://github.com/apache/ignite/pull/9084#discussion_r633357771



##########
File path: 
modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/exec/ExecutionServiceImpl.java
##########
@@ -614,6 +622,8 @@ private QueryPlan prepareDml(SqlNode sqlNode, 
PlanningContext ctx) throws Valida
         // Convert to Relational operators graph
         IgniteRel igniteRel = optimize(sqlNode, planner);
 
+        igniteRel = new FixDependentInsertNodeShuttle().visit(igniteRel);

Review comment:
       Currently it is a top-level node, but after IGNITE-12692 we could get an 
aggregate node on top of TableModify.




-- 
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:
[email protected]


Reply via email to