keith-turner commented on code in PR #5857:
URL: https://github.com/apache/accumulo/pull/5857#discussion_r2349538064


##########
test/src/main/java/org/apache/accumulo/test/functional/ZombieTServer.java:
##########
@@ -144,7 +145,24 @@ public static void main(String[] args) throws Exception {
     metricsInfo.init(MetricsInfo.serviceTags(context.getInstanceName(), 
"zombie.server",
         serverPort.address, ResourceGroupId.DEFAULT));
 
-    LockWatcher lw = new LockWatcher() {
+    final AtomicBoolean acquiredLock = new AtomicBoolean(false);
+
+    AccumuloLockWatcher lw = new AccumuloLockWatcher() {

Review Comment:
   Would also need to set the boolean to false in the other methods, so not 
sure the abstract class helps.  Every subtype would need to call the super 
method, which may be cumbersome and error prone.



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