You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/12/16 02:14:21 UTC

[GitHub] [pulsar-client-cpp] shibd commented on pull request #146: [fix] Use authoritative argument correctly in BinaryProtoLookupService::findBroker

shibd commented on PR #146:
URL: https://github.com/apache/pulsar-client-cpp/pull/146#issuecomment-1354083231

   > When a bundle is not loaded by any broker, a lookup request with authoritative as false will not trigger broker to load the bundle.
   
   @erobot Hi, Thanks for your pr. I don't quite understand this paragraph. According to the following code, when `authoritative` is false, an attempt is made to acquire new bundle ownership.
   
   https://github.com/apache/pulsar/blob/b669acc2dd341fd6281ee63dffa2554f4918f9a4/pulsar-broker/src/main/java/org/apache/pulsar/broker/namespace/NamespaceService.java#L485-L523


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