emkornfield commented on PR #12598: URL: https://github.com/apache/iceberg/pull/12598#issuecomment-2848467074
> LGTM. I think this is the correct balance to specify the current behavior from the reference implementation while not adding additional requirements to the spec. > > One nit comment is to add the details of this comment ([#12598 (comment)](https://github.com/apache/iceberg/pull/12598#discussion_r2067028228)) to the PR description. This was a learning for me and i think it would be good for future readers to reference. I added more details into the PR description, please let me know if there are any salient points you would like to see beyond what was added. -- 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
