wombatu-kun commented on PR #16548: URL: https://github.com/apache/iceberg/pull/16548#issuecomment-4532508098
@pvary thanks for the review! On the AI question: yes, this PR was prepared with AI assistance (drafting the fix, the refactor, and the test changes). Per the AI-assisted contribution guidelines I reviewed and verified it end-to-end — I understand the savepoint-completion race and ran the affected tests locally — and kept the wording and style aligned with the codebase. Happy to walk through any part during review. On applying it to all Flink versions: done in 806876ff1. I'd originally scoped it to v2.0/v2.1 because the observed CI failure was the `testStateRestore(File, ClusterClient)` variant that only exists there, but you're right that the same latent race lives in v1.20's `closeJobClient`. PR description is updated. -- 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]
