abhishek-chouhan commented on PR #19069: URL: https://github.com/apache/druid/pull/19069#issuecomment-3987595474
> Thanks for the review @abhishek-chouhan! This case is not covered. This would be more invasive as it requires merging BrokerServerView from broker1 and broker2. In this scenario I think it would be simpler to spin up an additional broker (e.g. druid/broker-admin) which watches all tiers and configure consoleBrokerService to point there. I do think we need the merging functionality to support the watchedTiers functionality in its entirety. While the workaround for broker works, its too heavy of a workaround just to make the console work with watchedTiers. This change is still valuable, we can take up the merging brokerServerView change as a followup enhancement. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
