ahmedabu98 commented on PR #39645: URL: https://github.com/apache/beam/pull/39645#issuecomment-5209865469
> there isn't currently a way to do accomplish the same behavior using IcebergIO I think the missing piece is a `withTableProperties(...)` option in IcebergIO. My recommendation based on effort level: 1. Simply recommend using Managed.ICEBERG, which enumerates more options like `table_properties` and `write_properties` 2. Add `withTableProperties(...)` to IcebergIO for parity Frankly I think `executionProperties` makes sense, but I wouldn't change the `writeProperties` naming, mostly because it aligns with the naming in Spark and Flink -- 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]
