[jira] [Resolved] (IMPALA-6908) IsConnResetTException() should include ECONNRESET

["Sailesh Mukil (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/IMPALA-6908?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sailesh Mukil resolved IMPALA-6908.
-----------------------------------
       Resolution: Fixed
    Fix Version/s: Impala 2.13.0

> IsConnResetTException() should include ECONNRESET
> -------------------------------------------------
>
>                 Key: IMPALA-6908
>                 URL: https://issues.apache.org/jira/browse/IMPALA-6908
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Distributed Exec
>    Affects Versions: Impala 2.10.0, Impala 2.11.0, Impala 3.0, Impala 2.12.0
>            Reporter: Michael Ho
>            Assignee: Sailesh Mukil
>            Priority: Major
>             Fix For: Impala 2.13.0
>
>
> {{IsConnReset()}} aims to check if the given exception is due to a stale connection. Apparently, it's missing the case in which the error code is ECONNRESET.
> {noformat}
> bool IsConnResetTException(const TTransportException& e) {
>   // Strings taken from TTransport::readAll(). This happens iff TSocket::read() returns 0.
>   // As readAll() is reading non-zero length payload, this can only mean recv() called
>   // by read() returns 0. According to man page of recv(), this implies a stream socket
>   // peer has performed an orderly shutdown.
>   return (e.getType() == TTransportException::END_OF_FILE &&
>              strstr(e.what(), "No more data to read.") != nullptr) ||
>          (e.getType() == TTransportException::INTERNAL_ERROR &&
>              strstr(e.what(), "SSL_read: Connection reset by peer") != nullptr);
> }
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)