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/06/05 13:52:02 UTC

[GitHub] [pulsar] Technoboy- opened a new pull request, #15940: [improve][broker] Make PulsarWebResource#getOwnerFromPeerClusterList async.

Technoboy- opened a new pull request, #15940:
URL: https://github.com/apache/pulsar/pull/15940

   ### Motivation
   
   PulsarWebResource#checkLocalOrGetPeerReplicationCluster is called by several methods, like `lookup`, `getPartitionedTopicMetadata`, `validateGlobalNamespaceOwnership`.  
   ![image](https://user-images.githubusercontent.com/6297296/172053930-2666d1b8-e542-4a3e-b38b-d458051eeb0f.png)
   
   These methods are all asynchronous and are called frequently in the system. However, I found that getOwnerFromPeerClusterList inside this method is indeed called synchronously, so I think this method needs to be changed to asynchronous.
   
   ### Documentation
   
   - [x] `doc-not-needed` 
   (Please explain why)
   


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


[GitHub] [pulsar] codelipenghui merged pull request #15940: [improve][broker] Make PulsarWebResource#getOwnerFromPeerClusterList async.

Posted by GitBox <gi...@apache.org>.
codelipenghui merged PR #15940:
URL: https://github.com/apache/pulsar/pull/15940


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