gustavodemorais commented on code in PR #28128:
URL: https://github.com/apache/flink/pull/28128#discussion_r3208870396
##########
flink-table/flink-table-api-java/src/main/java/org/apache/flink/table/api/Table.java:
##########
@@ -1431,6 +1431,17 @@ default TableResult executeInsert(
* INSERT-only row with a string {@code "op"} column indicating the
original operation (INSERT,
* UPDATE_AFTER, DELETE, etc.).
*
+ * <p>By default, the input is processed with row semantics (each row
independently). To
+ * co-locate rows with the same key in the same parallel operator
instance, partition the input
+ * first via {@link #partitionBy(Expression...)} and invoke {@link
+ * PartitionedTable#toChangelog(Expression...)}:
Review Comment:
nit: The new partitioned reference reads *"invoke
`PartitionedTable#toChangelog(Expression...)`"* — `fromChangelog` (line 1483)
has the same sentence with the *"with set semantics"* qualifier. Add it here
too for symmetry
--
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]
