HQebupt opened a new pull request, #17085: URL: https://github.com/apache/pulsar/pull/17085
### Motivation When the leader broker is closing, and it will release the ownership of its bundles first. The client will redo topic lookup , and this broker will handle it, resulting in unexpected results. For instance, some unassigned bundles maybe are taken ownership by itself again, but the broker can not server the bundles due to it is closing . see the screenshot log. Therefore, it is necessary to check the status of the broker when handling lookup requests. <img width="1909" alt="image" src="https://user-images.githubusercontent.com/4970972/184530824-b41d7a5b-7309-4251-af2f-58f97f1d58d3.png"> ### Modifications Check Pulsar Service state when handling lookup request. ### Verifying this change - [x] Make sure that the change passes the CI checks. This change is a trivial rework / code cleanup without any test coverage. ### Does this pull request potentially affect one of the following parts: If `yes` was chosen, please highlight the changes - Dependencies (does it add or upgrade a dependency): (no) - The public API: (no) - The schema: (no) - The default values of configurations: (no) - The wire protocol: (no) - The rest endpoints: (no) - The admin cli options: (no) - Anything that affects deployment: (no) ### Documentation Check the box below and label this PR (if you have committer privilege). Need to update docs? - [x] `doc-not-needed` -- 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]
