lhotari commented on issue #9216: URL: https://github.com/apache/pulsar/issues/9216#issuecomment-792537704
> @lhotari Thanks for the clarification. I don't think it should block the new feature implement. I understand your concern, if we merge more PRs related to the policies, we need to do more changes when fixing #9711. But for community cooperation, we should not block the new features. #9216 is planned to release in 2.8.0. So we can confirm before release 2.8.0, #9711 can be fixed, is this works for you? @codelipenghui Makes sense. Fixing #9711 might have broader impacts and dealing with it asap would be important. I hope it gets prioritized since it seems that the issue is cross-cutting and the current unsafe pattern is copied over to new feature development like we have seen in this case. There might be simple fixes to address some of the issues, for example replacing plain HashMaps with thread safe wrappers (Collections.synchronizedMap) or ConcurrentHashMaps, but that could leave some data consistency issues. ---------------------------------------------------------------- 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: [email protected]
