DanielLeens commented on PR #11503: URL: https://github.com/apache/seatunnel/pull/11503#issuecomment-5552672312
Follow-up on the `DorisErrorIT` race I flagged above: it reproduced on #11727 (head `a30a0bd7b`, which does not touch that test at all), so it is inherent to `dev`'s test rather than this branch's edit. Root cause: the test pins the frame `RecordBuffer.checkErrorMessageByStreamLoad`, but when the stopped container's network alias is already gone, the same `STREAM_LOAD_FAILED` is raised by `DorisStreamLoad.stopLoad` wrapping the `UnknownHostException` — a Docker teardown race. In both failing runs the error-code assertion passed and only the frame check failed. Source fix (test-only, contract unchanged): #12114 — asserts the failure surfaced from the sink's stream-load layer via either legitimate frame family. Once that lands, a `dev` sync clears it here; no change needed on this PR for it. -- 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]
