dlmarion opened a new issue, #5288:
URL: https://github.com/apache/accumulo/issues/5288

   **Describe the bug**
   ZooZap calls the 3 parameter version of `ServiceLock.deleteLock()` where the 
third version is the lock data (see below).
   
   
https://github.com/apache/accumulo/blob/7f8f12039df5ed410850d5e8743c68761ddb876d/server/base/src/main/java/org/apache/accumulo/server/util/ZooZap.java#L132-L153
   
   The third parameter is compared to the actual lock data in 
`ServiceLock.deleteLock`, which will never match the string `tserver`. I think 
the two parameter version of `ServiceLock.deleteLock` needs to be called in 
ZooZap so that it removes the lock data that it finds with no comparison.
   
   **Versions (OS, Maven, Java, and others, as appropriate):**
    - Affected version(s) of this project: 2.1.0+
   
   
   **Expected behavior**
   ZooZap should remove the tserver locks
   


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

Reply via email to