DanielLeens commented on PR #13040: URL: https://github.com/apache/seatunnel/pull/13040#issuecomment-6473737271
Thanks for the detailed follow-up. Keeping the catalog/sink-documentation change separate until #13023 lands is reasonable; the redundant assertion was optional. Since you have already rerun the failed unit-test job, please let that existing attempt finish rather than starting an overlapping run. I also need to qualify my earlier sync advice: the candidate HTTPS/REST test-port fixes #13028 and #13030 are still open, so syncing dev alone is not a demonstrated cure for the Hazelcast startup timeout. If that same job still fails, link its new result; no unrelated Databend dependency or module patch is requested. Gate snapshot (2026-10-28T15:51:57+00:00), head `51f068a1b4cadff094db8ea2cbae82d1aace3bb9`: [Build](https://github.com/apache/seatunnel/runs/122905859469) is `IN_PROGRESS`. An exact-head Build is already pending/in progress. Let that existing run finish; no overlapping rerun or source change is requested on the strength of branch drift. My [previous source conclusion](https://github.com/apache/seatunnel/pull/13040#pullrequestreview-5681309667) remains clear for the reviewed scope. CI is not being passed by inference, and this follow-up is not an effective approval. Comparison recorded in this scan: base `dev`, `diverged`, ahead 1 / behind 7, no merge conflict. Branch drift is metadata here and is not an additional sync request. -- 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]
