merlimat opened a new pull request, #25377:
URL: https://github.com/apache/pulsar/pull/25377
## Summary
- **ManagedLedgerTest**: Two test methods
(`testNonDurableCursorCreateForInactiveLedger` and
`testNoOrphanScheduledTasksAfterCloseML`) created local
`ManagedLedgerFactoryImpl` instances that were never shut down, leaking ~10
threads (OrderedScheduler + cache eviction executor). Fixed by adding
`@Cleanup("shutdown")`, consistent with other test methods in the same class.
- **PulsarMockBookKeeper**: `shutdown()` used `scheduler.shutdown()`
(graceful) which doesn't force thread termination. Changed to `shutdownNow()`
so the mock BK scheduler thread terminates immediately during test cleanup.
## 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]
