joseluisll commented on PR #8659: URL: https://github.com/apache/hadoop/pull/8659#issuecomment-5281670029
Cross-reference: #8682 (HADOOP-19964) complements this one. Setting a default `@Timeout` makes hung tests fail through JUnit rather than being killed at `forkedProcessTimeoutInSeconds`. That matters beyond keeping the class's results: it is also the only path on which an in-process listener still gets to run. The fork kill sends `Shutdown.KILL` and the fork executes `Runtime.halt()`, so nothing in-process gets a chance to report anything. #8682 restores `TimedOutTestsListener` — inert since the JUnit 5 migration, because it was a JUnit 4 `RunListener` wired through the Surefire `listener` property that the JUnit Platform provider ignores — as a JUnit Platform listener that prints a full thread dump with deadlock analysis when a test fails on timeout. With both in, a hung hdfs test fails at its own deadline *and* leaves a thread dump in `surefire-reports/*-output.txt`, where you can see what it was actually stuck on. One file in common, `hadoop-hdfs-project/hadoop-hdfs/pom.xml`: this PR adds inside `<systemPropertyVariables>`, #8682 removes the dead `<properties>` listener block just below it. Trial-merged clean in either order. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
