Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16687 )
Change subject: [client] KUDU-2966 configure connection negotiation timeout ...................................................................... [client] KUDU-2966 configure connection negotiation timeout This patch addresses KUDU-2966 for the Kudu C++ client, making the timeout for the connection negotiation configurable. This patch also contains a unit test for the newly introduced functionality. As for the motivation for this patch: we have observed a few clusters in the wild where connection negotiations were a bit slow when servers are under heavy load. It would be nice to control the connection negotiation timeout at the client side as well, so at least kudu CLI tool could use this functionality. The newly introduced functionality is to be used at least in kudu CLI (will be posted in a separate follow-up patch). Change-Id: I8573a572887be041637e28e518a7cd3a6d1f1693 Reviewed-on: http://gerrit.cloudera.org:8080/16687 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Andrew Wong <[email protected]> --- M src/kudu/client/client-internal.cc M src/kudu/client/client-internal.h M src/kudu/client/client-test.cc M src/kudu/client/client.cc M src/kudu/client/client.h M src/kudu/client/client_builder-internal.h M src/kudu/rpc/messenger.cc M src/kudu/rpc/messenger.h 8 files changed, 111 insertions(+), 34 deletions(-) Approvals: Alexey Serbin: Verified Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/16687 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8573a572887be041637e28e518a7cd3a6d1f1693 Gerrit-Change-Number: 16687 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
