[improve][broker][WIP] Add brokerId to lookup results and load manager data #22647
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WIP: This PR most likely requires a PIP and that will follow.
Motivation
In the past brokers didn't have a stable unique identifier. In PR #21894, the "lookupServiceAddress" was renamed to be "brokerId" so that there would be a single unique identifier for a broker in a cluster.
This PR continues making brokerId a first class identifier for a broker. The long term benefit is having consistency across the APIs. One reason to add this information to the topic lookup result is to be able to use the APIs in a way where you lookup a topic and receive that broker id. With this broker id, you could then lookup more details about the broker in the Admin API with the broker id.
Modifications
name
instead ofbrokerId
. Keep the namename
for backwards compatibilityDocumentation
doc
doc-required
doc-not-needed
doc-complete