jackye1995 commented on PR #6617: URL: https://github.com/apache/iceberg/pull/6617#issuecomment-1399009614
I think this PR is mostly ready to go. I see there is a comment in design doc from @flyrain: ``` "VERSION" is used in Iceberg to indicate any table changes including table property changes, which doesn't create a new snapshot. So versions and snapshots are mismatch. Can we call it "SNAPSHOT" to remove the ambiguity? ``` Requesting one more review from him -- 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
