AMashenkov commented on code in PR #2397:
URL: https://github.com/apache/ignite-3/pull/2397#discussion_r1291443954
##########
modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/rule/TableModifyConverterRule.java:
##########
@@ -71,8 +75,29 @@ protected PhysicalNode convert(RelOptPlanner planner,
RelMetadataQuery mq, Logic
IgniteTable igniteTable = relTable.unwrap(IgniteTable.class);
assert igniteTable != null;
+ IgniteDistribution distribution = igniteTable.distribution();
+
+ if (rel.getOperation() == Operation.DELETE) {
+ // To perform the delete, we need a row with key fields only.
+ // Input distribution contains the indexes of the key columns
according to the schema (i.e. for the full row).
+ // Here we adjusting distribution keys so that a row containing
only the key fields can be read.
+ List<Integer> keyFields = new ArrayList<>();
Review Comment:
Let's use IntList
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
