Wenzhe Zhou has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21900 )
Change subject: IMPALA-13427: Make connect timeout of statestore HA RPC tunable ...................................................................... IMPALA-13427: Make connect timeout of statestore HA RPC tunable In some deployment environment, unsuccessful socket connection attempts take long time to return error if connect timeout of the socket is set as 0. This causes statestored to tie up in unsuccessful connection attempts during initialization if its peer is not ready when statestore HA is enabled. Currently socket connect timeout of Thrift RPC is always set as 0. This patch makes socket connect timeout of Thrift RPC tunable, and adds a flag variable statestore_ha_client_rpc_conn_timeout_ms for the underlying socket connect timeout for Statestore HA RPC. Its default value equals 0, which equals the default value of TSocket.connTimeout_. Testing: - Added new test cases for statestore HA with non zero value for statestore_ha_client_rpc_conn_timeout_ms. - Passed core tests. Change-Id: Ie5840a76b4a34b4714c47b86f6366328b5ceed3a Reviewed-on: http://gerrit.cloudera.org:8080/21900 Reviewed-by: Norbert Luksa <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/rpc/thrift-client.h M be/src/runtime/client-cache.cc M be/src/runtime/client-cache.h M be/src/statestore/statestore.cc M tests/custom_cluster/test_statestored_ha.py 5 files changed, 68 insertions(+), 20 deletions(-) Approvals: Norbert Luksa: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/21900 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie5840a76b4a34b4714c47b86f6366328b5ceed3a Gerrit-Change-Number: 21900 Gerrit-PatchSet: 3 Gerrit-Owner: Wenzhe Zhou <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Norbert Luksa <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
