hangc0276 commented on pull request #10536: URL: https://github.com/apache/pulsar/pull/10536#issuecomment-838591537
> A good point was brought up by @linlinnn about the lack of thread-safety in the original implementation. I guess one solution would be to make both `handleSessionEvent` and `handleDataNotification` methods synchronized. @lhotari simply use synchronized will cause dead lock on close, i consider other solutions. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
