rdhabalia commented on PR #22841: URL: https://github.com/apache/pulsar/pull/22841#issuecomment-2149059191
> I understand, though the problem was introduced by masking the blocking call in the interface you introduced a while back. umm.. which interface? The issue is the interface in BookKeeper client which doesn't support async call and I don't recall any such interface in BookKeeper introduced by me. also, if we use ordered executor then it will not cause ordering. and anyway metadata calling thread and metadata callback threads will be always different so, I don't think it makes much difference for context switching. I agree we should improve mixing sync/async code path but there is also not any harm in supporting multiple callback threads model as it's not anything new which we are proposing. -- 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]
