kaisun2000 opened a new issue #1240: URL: https://github.com/apache/helix/issues/1240
Running TestCrushAutoRebalance in IntelligJ debugger, TestCrushAutoRebalance would clean up all threads created in `@AfterClass afterClass()`. But running mvn test under Helix-Core module, it would not invoke this afterClass and subsequent @afterclass in base class ZkTestBase The following log proves it: grep -e "has active threads cnt" -e "END " -e "START" out2.log TestExpandCluster has active threads cnt:137 START TestCrushAutoRebalance at Fri Aug 07 22:35:08 PDT 2020 START testZoneIsolationWithInstanceTag at Fri Aug 07 22:35:09 PDT 2020 END testZoneIsolationWithInstanceTag at Fri Aug 07 22:35:09 PDT 2020, took: 469ms. START testZoneIsolationWithInstanceTag at Fri Aug 07 22:35:09 PDT 2020 END testZoneIsolationWithInstanceTag at Fri Aug 07 22:35:10 PDT 2020, took: 503ms. START testZoneIsolationWithInstanceTag at Fri Aug 07 22:35:10 PDT 2020 END testZoneIsolationWithInstanceTag at Fri Aug 07 22:35:10 PDT 2020, took: 505ms. START testZoneIsolation at Fri Aug 07 22:35:10 PDT 2020 END testZoneIsolation at Fri Aug 07 22:35:11 PDT 2020, took: 458ms. START testZoneIsolation at Fri Aug 07 22:35:11 PDT 2020 END testZoneIsolation at Fri Aug 07 22:35:11 PDT 2020, took: 497ms. START testZoneIsolation at Fri Aug 07 22:35:11 PDT 2020 END testZoneIsolation at Fri Aug 07 22:35:12 PDT 2020, took: 470ms. START testLackEnoughLiveRacks at Fri Aug 07 22:35:12 PDT 2020 END testLackEnoughLiveRacks at Fri Aug 07 22:35:13 PDT 2020, took: 992ms. START testLackEnoughRacks at Fri Aug 07 22:35:13 PDT 2020 END testLackEnoughRacks at Fri Aug 07 22:35:14 PDT 2020, took: 1177ms. ---> note after class not running, no log of END TestCrushAutoRebalance from afterClass(), thus leaking 200+ threads. START TestCrushAutoRebalanceNonRack at Fri Aug 07 22:35:14 PDT 2020 START test at Fri Aug 07 22:35:14 PDT 2020 END test at Fri Aug 07 22:35:15 PDT 2020, took: 550ms. START test at Fri Aug 07 22:35:15 PDT 2020 END test at Fri Aug 07 22:35:15 PDT 2020, took: 549ms. START testWithInstanceTag at Fri Aug 07 22:35:15 PDT 2020 END testWithInstanceTag at Fri Aug 07 22:35:16 PDT 2020, took: 583ms. START testWithInstanceTag at Fri Aug 07 22:35:16 PDT 2020 END testWithInstanceTag at Fri Aug 07 22:35:16 PDT 2020, took: 590ms. START testLackEnoughLiveInstances at Fri Aug 07 22:35:16 PDT 2020 END testLackEnoughLiveInstances at Fri Aug 07 22:35:17 PDT 2020, took: 577ms. START testLackEnoughLiveInstances at Fri Aug 07 22:35:17 PDT 2020 END testLackEnoughLiveInstances at Fri Aug 07 22:35:17 PDT 2020, took: 566ms. START testLackEnoughInstances at Fri Aug 07 22:35:17 PDT 2020 END testLackEnoughInstances at Fri Aug 07 22:35:18 PDT 2020, took: 903ms. START testLackEnoughInstances at Fri Aug 07 22:35:18 PDT 2020 END testLackEnoughInstances at Fri Aug 07 22:35:19 PDT 2020, took: 968ms. END TestCrushAutoRebalanceNonRack at Fri Aug 07 22:35:20 PDT 2020 TestCrushAutoRebalanceNonRack has active threads cnt:362 ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
