clintropolis commented on issue #9507: optionally disable all of hardcoded zookeeper use URL: https://github.com/apache/druid/pull/9507#issuecomment-603005260 Some thoughts as I've been reviewing this (sorry I haven't finished yet): Do you view this as an interim configuration, to allow your work to proceed on an alternative discovery mechanism, until we can decouple zookeeper specific code from all of the places that need to check this setting? or is the plan to leave it like this? So far I find it kind of ugly to have a setting like this due to all of the if/else branches it causes, but maybe there is some obvious reason I haven't got to yet on why we aren't adding some sort of `druid.discovery.type=zk|none` instead of this enable/disable setting. I'll keep reviewing, and try to finish up later tonight.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
