You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/02/07 15:01:26 UTC

[GitHub] [pulsar] freeznet commented on pull request #9513: [Go Functions] Cache producers with sync.Map

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:
users@infra.apache.org