sungwy commented on PR #16498: URL: https://github.com/apache/iceberg/pull/16498#issuecomment-4530131188
Hi @wombatu-kun - thanks again for working on this PR. This looks good to merge. Just one last thing - this PR introduces a behavior change, and hence I think we should follow the guideline here and make sure we discuss this behavior change in the dev mailing list: https://iceberg.apache.org/contribute/#merging-pull-requests In short, I'd describe the rationale as to why we'd want this behavior change by referring to this issue @rdblue raised: https://github.com/apache/iceberg/issues/16493 And allow users to voice concerns regarding the behavior change first, before we move forward with the merge. -- 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]
