BewareMyPower commented on PR #8957: URL: https://github.com/apache/pulsar/pull/8957#issuecomment-1724773643
> To support SNI proxy routing, client needs two URL: sni-proxy-url + actual broker service URL as we have added in Java client library While I agree the implementation of this PR is different with the original PIP-60, the actual broker service URL is retrieved from the lookup response. https://github.com/apache/pulsar-client-cpp/blob/b242e1a0d8f316730ed5bfdd792c471ab2b6680c/lib/ClientImpl.cc#L537 The 2nd argument passed to `ConnectionPool::getConnectionAsync` is the `serviceUrl` in this PR. So in C++ client, users don't need to configure the actual broker service URL explicitly. Instead, this URL is returned according to the `advertisedListeners` or `advertisedAddress` config in 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
