raminqaf commented on code in PR #28235:
URL: https://github.com/apache/flink/pull/28235#discussion_r3311157138


##########
flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/api/Table.java:
##########
@@ -1459,9 +1459,19 @@ default TableResult executeInsert(
      *     descriptor("deleted").asArgument("op"),
      *     map("INSERT, UPDATE_AFTER", "false", "DELETE", 
"true").asArgument("op_mapping")
      * );
+     *
+     * // Opt out of full-delete semantics. The default (true) requires 
fully-populated DELETE
+     * // rows and inserts a ChangelogNormalize for upsert sources. When 
false, the function
+     * // emits partial DELETE rows: row semantics preserves the 
planner-derived upsert key
+     * // columns and nulls the rest; set semantics preserves the partition 
key. Requires an
+     * // upsert key or PARTITION BY; otherwise the call is rejected.

Review Comment:
   Done



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

Reply via email to