You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2019/11/19 15:56:36 UTC

[GitHub] [incubator-iceberg] aokolnychyi edited a comment on issue #652: Fix UUID check for existing tables without a UUID

aokolnychyi edited a comment on issue #652: Fix UUID check for existing tables without a UUID
URL: https://github.com/apache/incubator-iceberg/pull/652#issuecomment-555572328
 
 
   I think this PR would also resolve the problem when users upgraded to an Iceberg version that started assigning table UUIDs but their first operation wasn't a child of `SnapshotProducer` (e.g. update table properties). Such operations would write an explicit `null` as the table UUID that will corrupt the old logic.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org