[GitHub] [iceberg] wypoon commented on pull request #4809: Core: Recover the schema by reading previous metadata files

[GitBox <gi...@apache.org>]

wypoon commented on PR #4809:
URL: https://github.com/apache/iceberg/pull/4809#issuecomment-1131993501

   I don't think this is the way to do it. If the snapshot is written before schema ids were introduced, then most likely its ancestors also do not have schema ids, unless different versions of Iceberg are in use concurrently, and one is of a version before schema ids were introduced.
   I put up #3852 months ago to perform the recovery, and also added tests for the recovery. @kbendick can you please review?


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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