ctubbsii commented on code in PR #6470:
URL: https://github.com/apache/accumulo/pull/6470#discussion_r3554189847
##########
core/src/main/java/org/apache/accumulo/core/clientImpl/TabletLocator.java:
##########
@@ -116,7 +116,7 @@ public boolean equals(LocatorKey lk) {
private static final HashMap<LocatorKey,TabletLocator> locators = new
HashMap<>();
private static final HashMap<LocatorKey,OfflineTabletLocatorImpl>
offlineLocators =
new HashMap<>();
- private static boolean enabled = true;
+ private static volatile boolean enabled = true;
Review Comment:
Actually, it turns out this doesn't matter, since all accesses are
synchronized on the same object monitor. So this is redundant.
--
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]
