[GitHub] [accumulo] FineAndDandy opened a new issue #2459: TTimeoutTransport should apply timeout to Socket connection

[GitBox <gi...@apache.org>]

FineAndDandy opened a new issue #2459:
URL: https://github.com/apache/accumulo/issues/2459


   **Describe the bug**
   TTimeoutTransport.create(hostPort, timeoutMillis) should also apply the timeoutMillis to the Socket creation. As-is the code will wait an infinite amount of time to create the Socket connection and then apply timeoutMillis to the input and output stream only. 
   
   **Versions (OS, Maven, Java, and others, as appropriate):**
    - Affected version(s) of this project: 1.9, 2.0, 2.1
   
   **To Reproduce**
   Steps to reproduce the behavior (or a link to an example repository that reproduces the problem):
   1. Create a connection using TTimeoutTransport to a host that is waiting for connections but not completing the handshake, specifying a timeout of 2000, TTimeoutTransport.create(someHostAndPort, 2000);
   2. Observe that the call hangs beyond 2000ms
   
   **Expected behavior**
   It would make sense to also apply the timeoutMillis to the Socket creation. 


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] FineAndDandy commented on issue #2459: TTimeoutTransport should apply timeout to Socket connection

[GitBox <gi...@apache.org>]

FineAndDandy commented on issue #2459:
URL: https://github.com/apache/accumulo/issues/2459#issuecomment-1029222180


   I witnessed this while calling TabletServerBatchWriter.close() trying to write to a tserver that had gone into full GC and was later killed. The underlying call to socket.connect() hung indefinitely. 


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] ctubbsii closed issue #2459: TTimeoutTransport should apply timeout to Socket connection

[GitBox <gi...@apache.org>]

ctubbsii closed issue #2459:
URL: https://github.com/apache/accumulo/issues/2459


   


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] milleruntime commented on issue #2459: TTimeoutTransport should apply timeout to Socket connection

[GitBox <gi...@apache.org>]

milleruntime commented on issue #2459:
URL: https://github.com/apache/accumulo/issues/2459#issuecomment-1029215113


   Are you seeing connections hang somewhere? Which server type (tserver, gc, manager) are you seeing this affect?


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org