freeznet commented on pull request #9513: URL: https://github.com/apache/pulsar/pull/9513#issuecomment-774688710
hi @flowchartsman , thanks for your work on this issue. the `sync.Map` is LGTM, but seems not handling if cached producer been closed, do you think a cleanup work should be added when producer is no longer available? also, we should add some tests to verify this assumption. ---------------------------------------------------------------- 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]
