Technoboy- opened a new issue #8050: URL: https://github.com/apache/pulsar/issues/8050
In PulsarClientImpl, we use IdentityHashMap to hold the reference of producers/consumers like below: ``` producers.put(producer, Boolean.TRUE); ``` It's better to use List, for value is never used but stored. But Pulsar pursues performance, so looks good to use thread safe set -- Collections.newSetFromMap(new ConcurrentHashMap<>()) . ---------------------------------------------------------------- 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]
