dlmarion commented on code in PR #5662:
URL: https://github.com/apache/accumulo/pull/5662#discussion_r2180987954
##########
server/base/src/main/java/org/apache/accumulo/server/AbstractServer.java:
##########
@@ -112,9 +112,9 @@ protected AbstractServer(ServerId.Type serverType,
ConfigOpts opts,
if (advertHP.getHost().equals(ConfigOpts.BIND_ALL_ADDRESSES)) {
throw new IllegalArgumentException("Advertise address cannot be
0.0.0.0");
}
- advertiseAddress = advertHP;
+ advertiseAddress = new AtomicReference<>(advertHP);
} else {
- advertiseAddress = null;
+ advertiseAddress = new AtomicReference<>();
}
log.info("Bind address: {}, advertise address: {}", bindAddress,
advertiseAddress);
Review Comment:
Fixed in 92658d2
##########
server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java:
##########
@@ -207,8 +206,6 @@ public PausedCompactionMetrics getPausedCompactionMetrics()
{
private final BlockingDeque<ManagerMessage> managerMessages = new
LinkedBlockingDeque<>();
- volatile HostAndPort clientAddress;
-
private ServiceLock tabletServerLock;
private TServer server;
Review Comment:
Fixed in 92658d2
--
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]
