[GitHub] [pulsar-client-go] jonyhy96 commented on pull request #360: [Issue 257][feat] Support limit the retry number of reconnectToBroker

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

jonyhy96 commented on pull request #360:
URL: https://github.com/apache/pulsar-client-go/pull/360#issuecomment-699620996


   > > Thanks @jonyhy96 Here, I have only one confusion. When users use it, they need to set it on the producer and consumer at the same time, right? Can we consider exposing this parameter to the user in ClientOptions so that the user only needs to set it once
   > 
   > That would be more convenience for users to use, But the user loses more detailed control over the program.
   > By the way,do you mean add an option at [here](https://github.com/apache/pulsar-client-go/blob/master/pulsar/client.go#L77) and [here](https://github.com/apache/pulsar-client-go/blob/master/pulsar/client_impl.go#L40),in reconnectToBroker use p.clinet.maxReconnectToBroker to decide the retry logic?
   
   /cc @wolfstudy @shohi  
   
   Any further information about this? Maybe we can implement this feature before octuber 😄 


----------------------------------------------------------------
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:
users@infra.apache.org