merlimat opened a new pull request, #25375: URL: https://github.com/apache/pulsar/pull/25375
## Summary - The `SharedPulsarCluster` is a JVM-wide singleton (shut down via shutdown hook), so its ~90-100 broker/BK/client threads are not real leaks - The first `SharedPulsarBaseTest` subclass that triggers lazy initialization was being blamed for all those threads by the thread leak detector - Add `ThreadLeakDetectorListener.resetCapturedThreads()` to allow re-capturing the thread baseline after shared infrastructure is initialized - Call it from `SharedPulsarCluster.start()` so the shared cluster threads become part of the baseline This eliminates false-positive thread leak warnings from 16 test classes that extend `SharedPulsarBaseTest`. ## Documentation - [ ] `doc-required` - [x] `doc-not-needed` - [ ] `doc-complete` ## Matching PR in forked repository _none_ ### Documentation Checklist - [x] This PR does not require documentation changes --- > [!NOTE] > Ready to test _Labels: `area/test`_ -- 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]
