[GitHub] [pulsar-client-go] wolfstudy commented on a change in pull request #703: Fix stuck when reconnect broker

[GitBox <gi...@apache.org>]

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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org