maytasm commented on PR #12891: URL: https://github.com/apache/druid/pull/12891#issuecomment-3229458210
> > I have seen some cases where huge number of watches in ZooKeeper is causing performance issue. While there are improvements like #17482, I wonder if just switching this to `true` will be an easy win. I feel that most people are already using http segment discovery (it has been default since v25) but doesn't know about the existence of this flag. Hence, maybe switching the default of this flag to `true` will be beneficial to the community. I can raise a PR to switch the default to `true` unless there is any concern. > > We can also deprecate and remove zk segment discovery first before changing the default of this flag to make sure that all Druid clusters will be using http segment discovery. WDYT? > > IMO, setting the default to `true` would be good. I don't think there are big backwards compatibility concerns anymore, since http segment discovery has been supported for a while. We would just need a release note. PR: https://github.com/apache/druid/pull/18445 -- 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...@druid.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org
