mattfaltyn commented on PR #1952: URL: https://github.com/apache/iceberg-go/pull/1952#issuecomment-5490189247
> Please rebase > > Minor — add a not-found regression test > > If newest-containing is the best option available, a short comment saying so would help the next reader. Thanks for the thoughtful review! Addressed all three: rebased onto current `main`, added the not-found regression test, and documented that `FileScanTask` does not retain the equality delete’s sequence number, so the newest schema containing the complete key is the best available choice. All local and GitHub checks are green. -- 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]
