diff options
author | Yuxuan 'fishy' Wang <yuxuan.wang@reddit.com> | 2021-04-27 19:56:58 -0700 |
---|---|---|
committer | Yuxuan 'fishy' Wang <fishywang@gmail.com> | 2021-04-28 18:49:58 -0700 |
commit | fe3f8a1279660b2e3a47798fa2c8a7bd6800e4e9 (patch) | |
tree | 04a582b268b2ac6d01d128850a42e398dabaff74 /CHANGES.md | |
parent | 80415f05d621c861aeaa791fa93a1c3bee8b84bc (diff) | |
download | thrift-fe3f8a1279660b2e3a47798fa2c8a7bd6800e4e9.tar.gz |
THRIFT-5404: Allow other types of TTransportException to be timeouts
Client: go
Currently we only treat TTransportException with typeId == TIMED_OUT as
timeout (return true in Timeout function). When opening a new socket, if
we got a connect timeout from net.Dial, we wrap the error as
TTransportException with typeId == NOT_OPEN, thus it's no longer treated
as a timeout error.
Change the error to be directly wrapping the original error (instead of
recreate a new error with the same error message), and change
tTransportException.Timeout to also return true if the wrapped error
is a timeout error. This way we don't have to break anything (if code
rely on TTransportException.TypeId being NOT_OPEN in this case, that's
still true).
While I'm here, also update CHANGES.md from #2359.
Diffstat (limited to 'CHANGES.md')
-rw-r--r-- | CHANGES.md | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/CHANGES.md b/CHANGES.md index 99123ba7a..5853312e2 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -11,7 +11,8 @@ ### Go -- [THRIFT-5369](https://issues.apache.org/jira/browse/THRIFT-5369) - No longer pre-allocating the whole container (map/set/list) in compiled go code to avoid huge allocations on malformed messages +- [THRIFT-5369](https://issues.apache.org/jira/browse/THRIFT-5369) - TConfiguration.GetMaxMessageSize() now also applies to container sizes in TProtocol implementations provided +- [THRIFT-5404](https://issues.apache.org/jira/browse/THRIFT-5404) - TTransportException.Timeout would correctly return true when it's connect timeout during TSocket.Open call ## 0.14.1 |