ShivamSharma43 commented on PR #2523: URL: https://github.com/apache/iceberg-go/pull/2523#issuecomment-6805486627
Thanks @zeroshade, both points are addressed in a991bb1: 1. Added a bullet under "Unreleased compatibility notes" in `website/src/releases.md`: `DeleteColumn` on an identifier column (or a struct containing one) now fails in `Apply`, and `SetIdentifierField` is the way to force the delete. 2. Deletes are now checked against the original schema like Java (details in the inline thread), with a delete-then-add-same-name case in `TestDeleteIdentifierColumn`. I updated the description to match. -- 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]
