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



##########
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:
       At present, I see that you have added the option of MaxReconnectToBroker 
to the relevant test cases. This may not be very user-friendly. We need to 
ensure that the interface is as simple as possible.




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