Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1409. Make krpc call timeouts more resistant to process or reactor pauses ......................................................................
KUDU-1409. Make krpc call timeouts more resistant to process or reactor pauses In stress testing Impala on Kudu I've seen various RPC timeouts that turn out to be due to pauses on the client side. In particular, scenarios like IMPALA-2800[1] can cause the memory allocator inside Impala to block for several seconds, and that might cause us to think we missed a timeout. This adds a little workaround in our libev timeout handling code. The full description of the workaround can be found in connection.cc. [1] https://issues.cloudera.org/browse/IMPALA-2800 Change-Id: I7bff0bc1573a059f12be8bd3f46e301275e78392 Reviewed-on: http://gerrit.cloudera.org:8080/2745 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- M src/kudu/rpc/connection.cc M src/kudu/rpc/connection.h M src/kudu/rpc/messenger.h M src/kudu/rpc/rpc-test-base.h M src/kudu/rpc/rpc-test.cc M src/kudu/rpc/rpc_stub-test.cc 6 files changed, 103 insertions(+), 6 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2745 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7bff0bc1573a059f12be8bd3f46e301275e78392 Gerrit-PatchSet: 6 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
