Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8892 )
Change subject: KUDU-2236: fix flake in TestKuduClient ...................................................................... KUDU-2236: fix flake in TestKuduClient TestKuduClient.testCloseShortlyAfterOpen would expect 0 log spew, and ensure that no unexpected exceptions were caught. It noted that "connection disconnected" errors were possible, but added a wait to make them less likely. This turned out to be extremely flaky. This patch replaces the wait with logic to catch the appropriate error message. Without this patch, the test only passed 57/100 runs. With the patch, it passes 100/100. Change-Id: Ice5b404107dd425eab5ea3f382090b2304ef8a5e Reviewed-on: http://gerrit.cloudera.org:8080/8892 Reviewed-by: Hao Hao <[email protected]> Tested-by: Kudu Jenkins --- M java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java 1 file changed, 6 insertions(+), 4 deletions(-) Approvals: Hao Hao: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8892 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ice5b404107dd425eab5ea3f382090b2304ef8a5e Gerrit-Change-Number: 8892 Gerrit-PatchSet: 4 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins
