Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15196 )
Change subject: [java] Use Assert.assertThrows ...................................................................... [java] Use Assert.assertThrows This patch converts all usage of JUnit expected exception rules to `Assert.assertThrows`. This is the new way to test expected exceptions in JUnit 4.13+. Change-Id: I1959d6a33bfd634f01332e3c3ef4d7af77b60fb0 Reviewed-on: http://gerrit.cloudera.org:8080/15196 Tested-by: Kudu Jenkins Reviewed-by: Grant Henke <[email protected]> --- M java/kudu-client-tools/src/test/java/org/apache/kudu/mapreduce/tools/ITImportParquetPreCheck.java M java/kudu-client/src/test/java/org/apache/kudu/TestColumnSchema.java M java/kudu-subprocess-echo/src/test/java/org/apache/kudu/subprocess/echo/TestEchoSubprocess.java M java/kudu-subprocess/src/test/java/org/apache/kudu/subprocess/TestMessageIO.java 4 files changed, 82 insertions(+), 69 deletions(-) Approvals: Kudu Jenkins: Verified Grant Henke: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/15196 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1959d6a33bfd634f01332e3c3ef4d7af77b60fb0 Gerrit-Change-Number: 15196 Gerrit-PatchSet: 5 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
