cshannon opened a new issue, #3153: URL: https://github.com/apache/accumulo/issues/3153
As part of the investigation for #3134 I noticed another constructor [argument](https://github.com/apache/thrift/blob/b39370ec3bc96d201bbc82fbde136f98ae605ed1/lib/java/src/main/java/org/apache/thrift/transport/TNonblockingServerSocket.java#L83) for `clientTimeout` that can be configured as part of `TNonblockingServerSocket`. Currently we don't set this at all (so it defaults to 0) and in the fix in #3134 it will be set to 0 so we preserver the previous behavior. Setting to 0 means no timeout. We do have a [property](https://github.com/apache/accumulo/blob/c31194740080a1f12c65aec82fe46731f9da2069/core/src/main/java/org/apache/accumulo/core/conf/Property.java#L217) for RPC timeouts so we should investigate to see if it makes sense to use that property and set it to the clientTimeout value in the constructorinstead of leaving it as 0 and no timeout. -- 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]
