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

["Michael Ho (JIRA)" <ji...@apache.org>]

Michael Ho created IMPALA-6908:
----------------------------------

             Summary: 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.11.0, Impala 2.10.0, Impala 3.0, Impala 2.12.0
            Reporter: Michael Ho
            Assignee: Sailesh Mukil


{{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)