Hello Michael Ho, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9186 to look at the new patch set (#2). Change subject: IMPALA-6456: Add flags to configure rpc_negotiation_timeout_ms and negotiation thread count in KRPC ...................................................................... IMPALA-6456: Add flags to configure rpc_negotiation_timeout_ms and negotiation thread count in KRPC With the fix for KUDU-2228, the FLAGS_rpc_negotiation_timeout_ms was retired in KRPC. This patch introduces a flag to be able to configure that from the Impala side (FLAGS_rpc_negotiation_timeout_ms). It also introduces a flag to configure the negotiation thread count (FLAGS_rpc_negotiation_thread_count). Added a test to verify that setting FLAGS_rpc_negotiation_timeout_ms to 0 causes negotiation failures. We unfortunately can't write a test to check the same for FLAGS_rpc_negotiation_thread_count due to DCHECKS present in the code. Change-Id: I108d700e7eac04b678e21a3a920aac81ba8eede5 --- M be/src/rpc/rpc-mgr-test.cc M be/src/rpc/rpc-mgr.cc 2 files changed, 31 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/86/9186/2 -- To view, visit http://gerrit.cloudera.org:8080/9186 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I108d700e7eac04b678e21a3a920aac81ba8eede5 Gerrit-Change-Number: 9186 Gerrit-PatchSet: 2 Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Michael Ho <k...@cloudera.com>