ddanielr commented on PR #6166:
URL: https://github.com/apache/accumulo/pull/6166#issuecomment-3994707252
> Which test was failing?
`BigRootTabletIT.java` and `AdvertiseAndBindIT.java` were the two I noticed
failing, but any test that stopped and started the cluster should have been
failing.
> Was there a problem with the ZooZap code?
The ZooZap code was incorrect. #6077 replaced the use of
`removeSingletonLock` with a new method in ServiceLock.
`removeSingletonLock` calls `zapDirectory` first which takes a path and only
calls `recursiveDelete` for each child of the path.
The new ServiceLock code was calling `recursiveDelete` directly with the
top-level path. So it was removing all the gc locks, but then also removing the
path node `gc/lock` so new gc processes could not find the gc/lock path node so
all new locks were failing.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
