Alexey Serbin has submitted this change and it was merged. Change subject: [linked list test] use timeout for WaitAndVerify in all modes ......................................................................
[linked list test] use timeout for WaitAndVerify in all modes Constrain LinkedListTester::WaitAndVerify by the specified timeout for FINISH_WITH_SNAPSHOT_SCAN mode as well. Otherwise the TestLoadAndVerify scenario of the LinkedListTest test can run indefinitely under some conditions. One of such conditions is the expiration of the client authn token during the test: if that happens, WaitAndVerify() call might end up doing indefinite retries. The proper client behavior on the expiration of the authn token will be addressed in a separate changelist. Change-Id: If129738fbeef5394dfd1bcb160121e1d4131b3cc Reviewed-on: http://gerrit.cloudera.org:8080/6155 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> Reviewed-by: Mike Percy <[email protected]> --- M src/kudu/integration-tests/linked_list-test-util.h 1 file changed, 15 insertions(+), 14 deletions(-) Approvals: Mike Percy: Looks good to me, approved Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6155 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: If129738fbeef5394dfd1bcb160121e1d4131b3cc Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]>
