dlmarion commented on code in PR #4880:
URL: https://github.com/apache/accumulo/pull/4880#discussion_r1758714522
##########
core/src/main/java/org/apache/accumulo/core/rpc/clients/TServerClient.java:
##########
@@ -113,6 +129,9 @@ default Pair<String,C> getThriftServerConnection(Logger
LOG, ThriftClientTypes<C
TTransport transport =
context.getTransportPool().getTransport(type,
tserverClientAddress, rpcTimeout, context,
preferCachedConnections);
C client = ThriftUtil.createClient(type, transport);
+ if (type == ThriftClientTypes.CLIENT && preferredClientHost !=
null) {
Review Comment:
I don't think it should fall back, as the property is explicitly set. This
is why I added logging when the connection is successful. I'm going to add
logging for the failures too.
--
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]
