aloyszhang opened a new pull request #13055:
URL: https://github.com/apache/pulsar/pull/13055


   
   Fixes #13028
   
   ### Motivation
   
   For now, the result of the `lookup` command for a topic not existing by the 
rest API will return a brokerUrl.
   This pull request resolves this problem.
   
   ### Modifications
   Add topic exist check before get the brokerUrl
   
   ### Verifying this change
   
   This change added tests and can be verified as follows:
   HttpTopicLookupv2Test.testLookupTopicNotExist
   
   
   ### Documentation
   
   Check the box below and label this PR (if you have committer privilege).
   
   Need to update docs? 
   
   - [ ] `doc-required` 
     
     (If you need help on updating docs, create a doc issue)
     
   - [ ] `no-need-doc` 
     
     (Please explain why)
     
   - [ ] `doc` 
     
     (If this PR contains doc changes)
   
   
   


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


Reply via email to