[GitHub] [pulsar-client-go] ksankeerth commented on issue #928: [feature request] Client support set ServiceUrlProvider

["ksankeerth (via GitHub)" <gi...@apache.org>]

ksankeerth commented on issue #928:
URL: https://github.com/apache/pulsar-client-go/issues/928#issuecomment-1427797770

   > > Hi @shibd,
   > > I can take this issue and send a PR. Will you be able to review? Thanks
   > 
   > Great! This API is being discussed in the community, so follow this stack: https://lists.apache.org/thread/cojz6qxpc7v0j8vy0w476ttt0f89zr3z
   
   Hi @shibd,
   
   Thanks for pointing me to the discussions. I went through old Java PR and discussions. What I understood is that there are some new suggestions to get rid of `ServiceUrlProvider`.
   
   As per last comment from the thread,
   
   >> Oh, thanks Baodi - now I understand what he meant by "calling rebuild".
   I agree, it's not very intuitive.
   >> 
   >> Maybe?
   
   >> client.update(ConnectionInfo.builder()
   .setServiceUrl()
   .set...()
   .build())
   
   We are going to expose `client.update()` method to users. Detecting serviceURL's change is also user's responsibility. When they detect and change it, they have to re-build a new client by calling `client.update(ConnectionInfo.builder()` 
   
   Is my understanding correct? 
   
   Thanks!
   
   
   
   


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