merlimat commented on PR #22853: URL: https://github.com/apache/pulsar/pull/22853#issuecomment-2151024973
> was not good where we ignore actual solution without proper review and other one was merged even if that approach was incorrect. ?!? It was so much ignored that you had 2 people jumping on your issue report within 2 hours, discussing multiple ways on how to best solve the issue. > I am not sure how this can help fo fix strict affinity and deadlock issue. I think I've explained in the description and subsequent comment. It's similar to your approach, though in a more general way, it addresses all the usages where we are constructing the BK client, instead of just in one single spot. > and other one was merged even if that approach was incorrect. I think you have not read my multiple comments on why that PR was needed. > and please advise if anyone has plan to create PR so, one avoid doing duplicate efforts. ?? You posted your PR at the same time at the issue. I didn't fully agree with your proposed solution, so it's why I'm proposing a similar, though more general, way to fix the issue. I honestly don't understand how can you be polemic about this. It baffles me. -- 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]
