Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10265 )
Change subject: IMPALA-6908: IsConnResetTException() should include ECONNRESET ...................................................................... IMPALA-6908: IsConnResetTException() should include ECONNRESET The utility function IsConnResetTException() attempted to match error strings from RPCs that fail due to the remote end resetting the connection for any reason. However, it did not take care of the situation where ECONNRESET was sent on a cluster not using TLS. This patch adds this case as well and adds a custom cluster fault injection test to test the same. Change-Id: I1bb997a833917e5166c9ca192da219f50f4439e2 Reviewed-on: http://gerrit.cloudera.org:8080/10265 Reviewed-by: Sailesh Mukil <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/rpc/thrift-util.cc M be/src/testutil/fault-injection-util.cc M be/src/testutil/fault-injection-util.h M tests/custom_cluster/test_rpc_exception.py 4 files changed, 31 insertions(+), 18 deletions(-) Approvals: Sailesh Mukil: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/10265 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: 2.x Gerrit-MessageType: merged Gerrit-Change-Id: I1bb997a833917e5166c9ca192da219f50f4439e2 Gerrit-Change-Number: 10265 Gerrit-PatchSet: 5 Gerrit-Owner: Sailesh Mukil <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]>
