merlimat opened a new pull request, #25376: URL: https://github.com/apache/pulsar/pull/25376
## Summary Fix resource leaks detected by the thread leak detector across multiple test classes and one production class: - **BrokerEndpointsAuthorizationTest, ClusterEndpointsAuthorizationTest, TenantEndpointsAuthorizationTest**: `nobodyAdmin` PulsarAdmin was never closed in cleanup, leaking HTTP client threads (~12/12/8 threads) - **ClientInterruptTest**: Dedicated `PulsarClient` created in `@BeforeClass` was never closed in `@AfterClass` (~8 threads) - **PrometheusMetricsProviderConfigTest**: `testValidConfig()` started a provider but never stopped it, leaking Jetty server threads (~8 threads) - **KinesisSink** (production code fix): `scheduledExecutor` created in `open()` was never shut down in `close()`, leaking a scheduled thread pool (~9 threads) - **RedisSinkTest**: `RedisSink` was opened but never closed after the test, leaking the flush executor (~11 threads) ## 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]
