leventov commented on issue #6702: Add SelfDiscoveryResource URL: https://github.com/apache/incubator-druid/pull/6702#issuecomment-449077624 @kaijianding originally I created this endpoint to be used on brokers only, but then generalized to other node types, because why not. What do you think if I change the code so that the new endpoint is added only on brokers and coordinators for now? Unfortunately #6683 seems to be far from merging, and the next Druid release process should be started between 10th and 20th of January. I think it would be useful to add the new endpoint at least on brokers and coordinators. > Is it better to add similar listener mechanism when DruidNodeAnnouncer.announce(DiscoveryDruidNode) is called? I didn't understand what could be added when that method is called. Could you please elaborate?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
