codelipenghui opened a new pull request, #15633:
URL: https://github.com/apache/pulsar/pull/15633
### Motivation
#14839 fixes the HTTP lookup with multiple advertised listeners, and the
added test
is trying to verify the advertised listeners with 2 brokers, but the config
of the
the additional broker will not update, so the test gets passed.
The root cause is we are using the (advertised address + WebService port) to
register
the broker under the loadbalance path, but while creating the lookup result,
we will
use the WebService URL to get the data, so we will get
`MetadataStoreException$NotFoundException`.
### Modification
- Added properties support for ResourceUnit, so that we can carry more
information while getting the least loaded broker
- For creating the lookup result, use the broker znode name to get the
broker data if it presents in the ResourceUnit
### Verification
No new tests are needed, just fix the test that was added in #14839
### Documentation
Check the box below or label this PR directly.
Need to update docs?
- [x] `no-need-doc`
(bug fix)
--
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]
