nicoloboschi opened a new pull request, #15040: URL: https://github.com/apache/pulsar/pull/15040
### Motivation The pull https://github.com/apache/pulsar/pull/14619 causes deadlock while deleting namespace. The issue is 100% reproducible locally in the branch-2.8 if you run tests like * `ProxyAuthorizationTest` * `AdminApiTest#testCreateAndDeleteNamespaceWithBundles` The issue is how the completableFuture instances are chained. The original pull uses async methods to cleanup the ZK resources instead the porting (https://github.com/apache/pulsar/pull/14619). This causes deadlock on the `metadata-store` thread-pooll ### Modifications * Revert https://github.com/apache/pulsar/pull/14619 * Use async methods to cleanup the ZK resources An alternative could be to port the refactoring of the ZK cleanup with https://github.com/apache/pulsar/pull/11791 and https://github.com/apache/pulsar/pull/12711 but it's a more complicated solution (the cherry-picks don't apply cleanly) - [x] `no-need-doc` -- 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]
