[IMPALA-6159] DataStreamSender should transparently handle some connection reset by peer

• Type: Sub-task
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: Impala 2.12.0
• Fix Version/s: Impala 3.3.0
• Component/s: Distributed Exec
• Epic Color: ghx-label-4
• Assignee: Michael Ho
• Reporter: Michael Ho
• Created: 05/Nov/17 05:39
• Updated: 09/Jan/20 23:34
• Resolved: 09/Jan/20 23:34

Description

A client to server KRPC connection can become stale if the socket was closed on the server side due to various reasons such as idle connection removal or remote Impalad restart. Currently, the KRPC code will invoke the callback of all RPCs using that stale connection with the failed status (e.g. "Connection reset by peer"). DataStreamSender should pattern match against certain error string (as they are mostly output from strerror()) and retry the RPC transparently. This may be also be useful for KUDU-2192 which tracks the effort to detect stuck connection and close them. In which case, we may also want to transparently retry the RPC

FWIW, KUDU-279 is tracking the effort to have a cleaner protocol for connection teardown due to idle client connection removal on the server side. However, Impala still needs to handle other reasons for a stale connection.

Issue Links

• relates to: KUDU-279 RPC fails if it is sent exactly as the keepalive timeout expires
• relates to: KUDU-2192 KRPC should have a timer to close stuck connections