wombatu-kun commented on PR #17925: URL: https://github.com/apache/iceberg/pull/17925#issuecomment-5612393101
Keep it here - the diff no longer overlaps #17713, which touches only `Channel.java` and its own test, so folding it in would tie a self-contained `CommitState` fix to that PR's open consolidation question. The one to reconcile with instead is #17080: it edits `CommitState.java` and `TestCommitState.java` to filter `readyBuffer` by commit id inside `validThroughTs`, it was closed by the stale bot without ever being reviewed, and issue #17077 is still open. -- 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]
