SEPURI-SAI-KRISHNA commented on PR #13115: URL: https://github.com/apache/seatunnel/pull/13115#issuecomment-6624708973
@DanielLeens thanks, you were right about Issue 1. It is fixed in the new commit: `setUp` keeps the `URLClassLoader` in a field and `tearDown` closes it, in both copies of `SinkExecuteProcessorTest`. No production change. I have no Windows machine here, so I checked it on Linux with a throwaway test in both starter modules that lists the files the JVM holds open under the temp directory: ``` before close: [connectors/connector-demo.jar] after close: [] ``` That matches the Windows log, where `connectors` could not be deleted but `plugins` could. The real proof is the Windows jobs in the new fork run started by this push, which has not finished yet. It is also the first time the 1.13 copy runs on Windows, because the old run stopped at the common module. On the other two jobs in the old run: - `unit-test (11, ubuntu-latest)` is as you described: `JobMasterTest` timed out in `seatunnel-engine-server` and the Flink starter modules were skipped. - `unit-test (17, ubuntu-latest)` did run the new test, and it passed in both modules. The log then stops with no build result. I did not find `No space left on device` in the job log itself, so I cannot confirm the cause from there. -- 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]
