szehon-ho commented on PR #10020: URL: https://github.com/apache/iceberg/pull/10020#issuecomment-2021611486
Redid the approach, now I re-assign partition field ids. I fix the initial problems, it requires a bit of finesse. The broad picture: - When we need to read manifests, we need to use the original specs. This is because it needs the original field ids to deserialize the manifest entry, which has those partition field ids. - When we need to bind filters (like for predicate eval), we need the new spec with reassigned ids, because it needs to bind against the updated schema. -- 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
