Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12321 )
Change subject: [java] fix ITClient test retrying ...................................................................... [java] fix ITClient test retrying ITClient is flaky due to KUDU-2390, but it's also unable to retry upon failure. I think this is caused by the static error latch which isn't reset in between test runs. There's no reason for it to be static; I'm guessing it's a holdover from when the minicluster state was static. I also added an @After to close the client instance used by the test, which will hopefully reduce the amount of log pollution in the retries. Change-Id: I4a9f6a27541de02bbc8cf9dc3ba72824a6324a08 Reviewed-on: http://gerrit.cloudera.org:8080/12321 Tested-by: Kudu Jenkins Reviewed-by: Grant Henke <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M java/kudu-client/src/test/java/org/apache/kudu/client/ITClient.java 1 file changed, 36 insertions(+), 25 deletions(-) Approvals: Kudu Jenkins: Verified Grant Henke: Looks good to me, approved Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/12321 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I4a9f6a27541de02bbc8cf9dc3ba72824a6324a08 Gerrit-Change-Number: 12321 Gerrit-PatchSet: 2 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
