abdessalems commented on PR #11727: URL: https://github.com/apache/seatunnel/pull/11727#issuecomment-5441639543
CI is back, @DanielLeens — all four unit-test legs completed and passed this time: 50/46/60/61 min, versus hitting the 90-minute cap on the last two heads. The root cause was the getClassLoaders() guard from the previous round, although the guard itself is correct. The test helper only registered a class loader for the first task in a group, so tests deploying two tasks left the second without one. The helper now mirrors deployTask() and registers one per task. Locally and in CI, the relevant tests are now green. I also ruled out the other two candidates: removing the synchronized block made no difference, while removing the recycleClassLoader call caused a hang. The only remaining red job is all-connectors-it-1 (8, ubuntu-latest), which was already failing on 51c6a46a before these changes and is unrelated to seatunnel-engine-server. kudu-connector-it was cancelled behind it. @waterWang — also worth flagging for #11757: its unit-test job was skipped, so this issue wouldn't have been caught there either. -- 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]
