Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12043 )
Change subject: build: enable Java flaky test reporting ...................................................................... Patch Set 8: > > > Currently we retry all Java tests in dist_test.py. Should we enable flaky > > > retry functionality like the C++ tests? This could be done in this patch > > > or a follow on. > > > > Yeah potentially, though I need to page back in how we retry on the C++ > > side. It used to be based on the "list of flaky tests" but has evolved > > somewhat since. > > The premise for not always retrying the C++ tests is the assumption that we > should only retry "known flaky" tests, to have a higher chance of catching a > commit causing flakiness at code review time. Only nightly runs against > master and regular loops on "known flaky" tests contribute to a test's flaky > percentage (if it's > 0%) I've gone ahead and done this in a follow-on patch. Need to test, then will publish. -- To view, visit http://gerrit.cloudera.org:8080/12043 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifef74fc9bf5453105c267418fa24daf4c33f73f3 Gerrit-Change-Number: 12043 Gerrit-PatchSet: 8 Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com> Gerrit-Comment-Date: Tue, 02 Apr 2019 04:59:15 +0000 Gerrit-HasComments: No
