fishy commented on pull request #2181:
URL: https://github.com/apache/thrift/pull/2181#issuecomment-723397818


   >Another reason is that currently the only reason cancels the context is the 
disconnection
   
   This is not true. The feature this change implemented is mainly for clients 
to use (servers don't care about read timeouts unless they don't intent to 
support long connections/client pools). It's very common for clients to set a 
timeout in the context object.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to