Technoboy- commented on PR #21015: URL: https://github.com/apache/pulsar/pull/21015#issuecomment-1682246467
> Thanks for being so quick with providing a fix 👍 The only question I had when looking at the code is that `getLookupServiceAddress()` was used in the `BrokerRegistry` to set the broker id, which will change now - Does this mean when we apply the change that there is stale metadata that has to be cleaned up? No need to clean-up -- 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-unsubscr...@pulsar.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org