wolfstudy commented on a change in pull request #703:
URL: https://github.com/apache/pulsar-client-go/pull/703#discussion_r782681463
##########
File path: pulsar/internal/connection.go
##########
@@ -163,6 +163,9 @@ type connection struct {
consumerHandlersLock sync.RWMutex
consumerHandlers map[uint64]ConsumerHandler
+ reconnectFlagLock sync.Mutex
+ reconnectFlag bool
Review comment:
Based on the current wrapper implementation of Connection and
ConnectionPool, we have two elegant ways to handle this:
1. In my current implementation, a flag is introduced into the struct of
connection to mark the state of reconnection
2. When using changeState() to trigger handleCloseProducer, set the state of
the current connection to close
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
