Caideyipi opened a new pull request, #18246: URL: https://github.com/apache/iotdb/pull/18246
## Description Follow up on the flaky `SharedTsBlockQueueTest` cleanup by fixing similar concurrency-test issues: - replace an unbounded non-daemon polling thread in `ConcurrentIterableLinkedQueueTest` with a bounded executor task - replace fixed sleeps with latches that establish worker startup - retain submitted futures and propagate worker failures to the test thread - shut down every local executor in `finally` - replace preemptive JUnit timeouts in the WAL tests with bounded future waits that can clean up their workers This prevents background threads from leaking into later tests and makes failures report their underlying worker exception instead of timing out. ## Testing - `mvn -o -nsu -Ddevelocity.off=true test -pl iotdb-core/node-commons -Dtest=ConcurrentIterableLinkedQueueTest -DfailIfNoTests=false`: 24 tests passed - repeated `ConcurrentIterableLinkedQueueTest` 20 times: 20/20 passed - compiled and ran `WALNodeWaitForRollFileTest` directly: 7 tests passed, including the 30-second auto-roll path - `mvn -o -nsu -Ddevelocity.off=true spotless:check -pl iotdb-core/node-commons,iotdb-core/datanode` - `mvn -o -nsu -Ddevelocity.off=true checkstyle:check -pl iotdb-core/node-commons,iotdb-core/datanode` The full DataNode reactor remains blocked before the test phase by the existing generated Thrift `javax.annotation` compilation issue noted in #18244. -- 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]
