Guosmilesmile commented on code in PR #14414:
URL: https://github.com/apache/iceberg/pull/14414#discussion_r2516544478
##########
docs/docs/flink-writes.md:
##########
@@ -537,7 +544,9 @@ The Dynamic Iceberg Flink Sink is configured using the
Builder pattern. Here are
| `set(String property, String value)` | Set any Iceberg write
property (e.g., `"write.format"`, `"write.upsert.enabled"`).Check out all the
options here: [write-options](flink-configuration.md#write-options) |
| `setAll(Map<String, String> properties)` | Set multiple
properties at once
|
+
### Notes
- **Range distribution mode**: Currently, the dynamic sink does not support
the `RANGE` distribution mode, if set, it will fall back to `HASH`.
- **Property Precedence Note**: When conflicts occur between table properties
and sink properties, the sink properties will override the table properties
configuration.
+- **Table Format Version upgrade**: The dynamic sink does not support directly
upgrading the underlying table from V2 to V3 when position deletes are present.
Review Comment:
fix
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
