thrift-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yuxuan Wang (Jira)" <j...@apache.org>
Subject [jira] [Commented] (THRIFT-5214) go: Implement connection check in TSocket
Date Sun, 07 Jun 2020 23:44:00 GMT

    [ https://issues.apache.org/jira/browse/THRIFT-5214?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17127801#comment-17127801
] 

Yuxuan Wang commented on THRIFT-5214:
-------------------------------------

I created 3 PRs out of my debugging: #2172, #2173, & #2174. Only #2172 is needed to fix
the bug we saw that caused us to revert the change. The other 2 are just little improvements.

> go: Implement connection check in TSocket
> -----------------------------------------
>
>                 Key: THRIFT-5214
>                 URL: https://issues.apache.org/jira/browse/THRIFT-5214
>             Project: Thrift
>          Issue Type: Improvement
>          Components: Go - Library
>            Reporter: Yuxuan Wang
>            Priority: Major
>             Fix For: 0.14.0
>
>          Time Spent: 1.5h
>  Remaining Estimate: 0h
>
> The first issue described in [this GitHub engineering blog article|https://github.blog/2020-05-20-three-bugs-in-the-go-mysql-driver/]
is also an issue when we use thrift client pools. We implemented a thrift client pool by checking
client's TTransport.IsOpen before using that client, and also added (arbitrary) TTL to the
clients to avoid using clients that's been opened for too long, but we still occasionally
get broken pipes and unexpected EOF errors when using those clients. I think implementing
the same connection check described by that article in TSocket.IsOpen (and/or when try to
read 0 byte from TSocket.Read) would greatly help the situation.
> But there are a few limitations of the connection check implementation, and I'm not sure
how acceptable are they in the thrift library:
> 1. It only works on non-windows systems (so for windows we'll have to fallback to the
old IsOpen implementation, I guess?)
> 2. It requires go 1.9+, what's thrift library's minimal go version supported?



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Mime
View raw message