keith-turner commented on PR #5662:
URL: https://github.com/apache/accumulo/pull/5662#issuecomment-2997163832
Have not had a chance to look at this yet. Saw the following in the tserver
logs today when debugging a problem w/ an IT in main. Commenting in case this
is related to these changes.
```
TabletServer_22668134.out:2025-06-23T15:56:04,700 36 [tserver.TabletServer]
INFO : address = org.apache.accumulo.server.rpc.ServerAddress@43b6dd99
```
Made the following changes locally to get better info for this log message,
but not completely sure if the logging of the bind addr is correct. I needed
better logging to see what address the tserver was using because the manager
was having trouble contacting it.
```diff
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
index 1507b4bf9f..9f33778096 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
@@ -475,9 +475,10 @@ public class TabletServer extends AbstractServer
implements TabletHostingServer
TProcessor processor =
ThriftProcessorTypes.getTabletServerTProcessor(this, clientHandler,
thriftClientHandler,
scanClientHandler, thriftClientHandler, thriftClientHandler,
getContext());
- ServerAddress address = startServer(getBindAddress().toString(),
processor);
- updateAdvertiseAddress(address.address);
- log.info("address = {}", address);
+ var bindAddress = getBindAddress();
+ ServerAddress address = startServer(bindAddress, processor);
+ updateAdvertiseAddress(address.getAddress());
+ log.info("advertise address = {} bind address = {}",
address.getAddress(), bindAddress);
return address;
}
```
--
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]
