wolfstudy commented on a change in pull request #703:
URL: https://github.com/apache/pulsar-client-go/pull/703#discussion_r782672593



##########
File path: pulsar/internal/connection_pool.go
##########
@@ -75,13 +75,15 @@ func (p *connectionPool) GetConnection(logicalAddr 
*url.URL, physicalAddr *url.U
        p.Lock()
        conn, ok := p.connections[key]
        if ok {
-               p.log.Debugf("Found connection in pool key=%s logical_addr=%+v 
physical_addr=%+v",
+               p.log.Infof("Found connection in pool key=%s logical_addr=%+v 
physical_addr=%+v",
                        key, conn.logicalAddr, conn.physicalAddr)
 
-               // remove stale/failed connection
-               if conn.closed() {
+               // When the current connection is in a closed state or the 
broker actively notifies that the

Review comment:
       The action of the broker's notification to close the connection will be 
processed by the two commands CommandCloseProducer and CommandCloseConsumer. 
Generally, it may trigger the automatic load balancing of the topic or the 
unload operation will trigger the active closing of the broker.




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


Reply via email to