keith-turner opened a new issue #2082:
URL: https://github.com/apache/accumulo/issues/2082


   Compactors register their-self in ZK w/ a service lock.  Service locks are a 
persistent node w/ ephemeral nodes underneath.  If compactors come and go and 
different addresse+port, then this can pollute ZK w/ persistent host+port nodes 
w/o any children.  The Coordinator should periodically scan for persistent 
compactor nodes w/ no children and delete them.
   
   The manager does a similar thing for Tservers in the class LiverTserverSet, 
however the handling of this case in the manager is mixed in w/ lots of other 
things.  One thing worth taking from the Manager code is [ignoring the ZK 
NotEmptyException](https://github.com/apache/accumulo/blob/16a3cec36981a694ecdfbefb839ecf9ddd70a535/server/base/src/main/java/org/apache/accumulo/server/manager/LiveTServerSet.java#L280-L288)
 when trying to delete.  This a race condition that could occur in the case 
where between reading the children from ZK and deleting that something was 
created.


-- 
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]


Reply via email to