Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15174 )
Change subject: [rpc-test] fix flaky TestCancellation ...................................................................... [rpc-test] fix flaky TestCancellation KUDU-3042 added a test case for the interaction between rpc cancellation and rpc timeouts. This test was flaky as it assumed that rpcs that are cancelled will return with an aborted status, which is not always the case due to cancellation being best effort. The fix is to make the test more permissive by allowing the rpcs to fail with either an aborted or a timed out status. Change-Id: If140a9215c94d781c2c61ec30b18d2d67d80310a Reviewed-on: http://gerrit.cloudera.org:8080/15174 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/rpc/rpc-test-base.h 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/15174 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: If140a9215c94d781c2c61ec30b18d2d67d80310a Gerrit-Change-Number: 15174 Gerrit-PatchSet: 3 Gerrit-Owner: Thomas Tauber-Marshall <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
