tree 9680d1a399fe1e60a7f399a5a80cf3fb6a6a501a
parent 3f78c70bbba82c733b1545d39eb5b489ac384e4c
author Yuxuan 'fishy' Wang <yuxuan.wang@reddit.com> 1592025755 -0700
committer Yuxuan 'fishy' Wang <yuxuan.wang@reddit.com> 1592252918 -0700

THRIFT-5233: Handle I/O timeouts in go library

Client: go

As discussed in the JIRA ticket, this commit changes how we handle I/O
timeouts in the go library.

This is a breaking change that adds context to all Read*, Write*, and
Skip functions to TProtocol, along with the compiler change to support
that, and also adds context to TStandardClient.Recv, TDeserializer,
TStruct, and a few others.

Along with the function signature changes, this commit also implements
context cancellation check in the following TProtocol's ReadMessageBegin
implementations:

- TBinaryProtocol
- TCompactProtocol
- THeaderProtocol

In those ReadMessageBegin implementations, if the passed in context
object has a deadline attached, it will keep retrying the I/O timeout
errors, until the deadline on the context object passed. They won't
retry I/O timeout errors if the passed in context does not have a
deadline attached (still return on the first error).