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



##########
File path: pulsar/producer.go
##########
@@ -138,6 +138,9 @@ type ProducerOptions struct {
 
        // A chain of interceptors, These interceptors will be called at some 
points defined in ProducerInterceptor interface
        Interceptors ProducerInterceptors
+
+       // MaxReconnectToBroker set the maximum retry number of 
reconnectToBroker. (default: no retry)
+       MaxReconnectToBroker int

Review comment:
       @jonyhy96 Can we not modify the original default behavior, and we will 
always retry by default. When we set the maximum number of retries, we will 
turn off the retry logic after reaching the specified threshold.
   
   Because the current behavior will break the backward compatibility of the 
version




----------------------------------------------------------------
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:
[email protected]


Reply via email to