SEZ9 commented on PR #10583: URL: https://github.com/apache/seatunnel/pull/10583#issuecomment-5390957215
Thanks for the detailed status check, @DanielLeens — that answers everything I needed to know about where this stands. To confirm: yes, the failed `Build` run on `cf1aa3192c` (https://github.com/apache/seatunnel/runs/96302532851) is the Maven Central `aopalliance-repackaged` 502 flake you traced, and it's unrelated to this documentation-only change. Since the Javadoc content on this head is byte-for-byte identical to what you verified and approved at `98e0832aec2b75d7b0513080c177a7ca25e045aa`, no further content review is needed. On the retrigger: I'll trigger a fresh run from the maintainer side. If that doesn't take for any reason, the fallback is for the PR author to push an empty commit to this head, as you suggested. Concrete remaining asks: 1. A fresh green `Build` run on `cf1aa3192c` — I'll kick this off; PR author, please push an empty commit if it doesn't land shortly. 2. Nothing else — once CI is green on this exact head, this is ready to merge given your existing approval. Thanks again for the thorough re-verification and for keeping the CI trail so clear. <!-- streview-comment:499 --> -- 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]
