[GitHub] kaijianding opened a new pull request #6683: fix huge number of watches in zk issue

[GitBox <gi...@apache.org>]

kaijianding opened a new pull request #6683: fix huge number of watches in zk issue
URL: https://github.com/apache/incubator-druid/pull/6683
 
 
   fix #6647 .
   I guess the origin Announcer.java is better performance for segment announcement, so I just keep it.
   
   I guess the root cause of this issue is that each child path has a watch on zookeeper server even we only add listener at the parent path. When the path is like /druid/prod/announces/$host, the number of watches is n*n (each server watches every host including itself). Plus the path for lookup, the number is n*n*2.
   
   I test it in my product environment, the number of watches is reduced as expected.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org