pvary commented on PR #12691: URL: https://github.com/apache/iceberg/pull/12691#issuecomment-2782366843
@Guosmilesmile: I did a little digging, and TBH, I don't understand why are these configs handled differently than the standard `FlinkWriteOptions` and `FlinkReadOptions`. Maybe we should ask @stevenzwu and @mxm and deprecated and migrate these configurations to there? My current feeling is that we should migrate them to the standard read/write options, deprecate the old ones, and document them only in their final place. But I might miss some historical details 😄 -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
