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/01/25 15:54:01 UTC

[GitHub] [pulsar-client-go] zzzming commented on issue #448: Access to partitionConsumer state not thread-safe.

zzzming commented on issue #448:
URL: https://github.com/apache/pulsar-client-go/issues/448#issuecomment-766913900


   It looks like this is a miss. In `state` was synchronized in producer_partition.go and connection.go with sync/atomic 
   
   https://github.com/uber-go/atomic/blob/master/uint32.go uses sync/atomic underneath. Can we converge to use uber library for simplicity? @merlimat @flowchartsman 


----------------------------------------------------------------
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