Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11037 )
Change subject: [Java] Retry tests that don’t inherit from BaseKuduTest ...................................................................... [Java] Retry tests that don’t inherit from BaseKuduTest The Scala tests and a couple special case tests don’t inherit from BaseKuduTest but may still be flaky. To fix this I moved the retry configuration logic into the RetryRule and added the rule to the a few tests directly: - TestSecurity - TestKuduMetastorePlugin Additionally I updated the Scala TestContext, now called KuduTestSuite, to use the JUnitSuite and include the RetryRule so that all of the Scala tests would be retried. This required a few syntax changes but has the added benefit of not needing `@RunWith(classOf[JUnitRunner])` at the top of each test class. It also follows similar syntax to the rest of our tests. Change-Id: I596fde91a13f7e644fd1b1814e9a672f96fa0e4b Reviewed-on: http://gerrit.cloudera.org:8080/11037 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins --- M java/kudu-backup/src/test/scala/org/apache/kudu/backup/TestKuduBackup.scala M java/kudu-client/src/test/java/org/apache/kudu/client/BaseKuduTest.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java M java/kudu-client/src/test/java/org/apache/kudu/junit/RetryRule.java M java/kudu-hive/build.gradle M java/kudu-hive/pom.xml M java/kudu-hive/src/test/java/org/apache/kudu/hive/metastore/TestKuduMetastorePlugin.java M java/kudu-spark-tools/src/test/scala/org/apache/kudu/spark/tools/ITBigLinkedListTest.scala M java/kudu-spark-tools/src/test/scala/org/apache/kudu/spark/tools/TestImportExportFiles.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/DefaultSourceTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduContextTest.scala M java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduRDDTest.scala R java/kudu-spark/src/test/scala/org/apache/kudu/spark/kudu/KuduTestSuite.scala 13 files changed, 196 insertions(+), 125 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/11037 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I596fde91a13f7e644fd1b1814e9a672f96fa0e4b Gerrit-Change-Number: 11037 Gerrit-PatchSet: 4 Gerrit-Owner: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Kudu Jenkins
