badalprasadsingh opened a new pull request, #1856: URL: https://github.com/apache/iceberg-go/pull/1856
### Description Fixes #1855, a long running issue, unnoticed till #1818. Previous to it, `snapshotProducer.spec(id)` returned an empty partition spec when a spec id was not registered in table metadata. Four callers relied on it. So, a delete-file carrying an unknown spec id committed happily into a manifest declaring the unpartitioned spec 0, while its entry still held a different partition tuple. That is corrupt metadata written with no error at all. As I said earlier, #1818 did spot this but it fixed the added-data-file path only. This moves the check into the helper itself so every path behaves the same way. ### Testing Added regression tests wherever required. -- 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]
