Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16886 )
Change subject: KUDU-2612: two more scenarios for txn keepalive in client ...................................................................... KUDU-2612: two more scenarios for txn keepalive in client This patch adds two more test scenarios related to txn keepalive messages automatically sent by Kudu C++ client. These are to verify the corresponding functionality of the client when TxnManager instances are not available for short and long time intervals, where 'short' and 'long' are relative to the txn keepalive heartbeat timeout interval. In addition, I sneaked in one small fix to avoid TSAN warnings when running the newly introduced tests: the removed CHECK() in Master::WaitForTxnManagerInit() isn't crucial. Change-Id: I42988996f1ddb1cd456d289ea7e15586bd7e3dc7 Reviewed-on: http://gerrit.cloudera.org:8080/16886 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> --- M src/kudu/client/client-test.cc M src/kudu/master/master.cc 2 files changed, 95 insertions(+), 1 deletion(-) Approvals: Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/16886 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I42988996f1ddb1cd456d289ea7e15586bd7e3dc7 Gerrit-Change-Number: 16886 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
