rdhabalia commented on PR #22841: URL: https://github.com/apache/pulsar/pull/22841#issuecomment-2148635308
> If you have N threads you can still have the same issue depending on which threads the requests are assigned. It might not give a guarantee for all scenarios but it helps in most of the scenarios to prevent the deadlock. Before Metadaastore implementation, we had multiple zk callback threads to avoid this issue which was working in most of the cases. we are wasting lot of time to keep our system stable with such issues. and this fix helps us to keep it stable in such regressions where extra thread helps to prevent race-condition and because of that avoid the deadlock. I don't see any harm to keep it multi threads for metadata-store and if you don't want to keep in all usecases then I can make it configurable from service configuration. -- 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]
