TestSecurity: remove errant Precondition Its presence makes it impossible to retry the tests in this class.
An alternative would be to null out miniCluster in tearDown(), but I don't think that should be necessary for creating a new cluster. Change-Id: I0ba826d3ec722cb20b35d4c34817d30ab9782976 Reviewed-on: http://gerrit.cloudera.org:8080/11325 Reviewed-by: Dan Burkert <danburk...@apache.org> Tested-by: Kudu Jenkins Reviewed-by: Grant Henke <granthe...@apache.org> Project: http://git-wip-us.apache.org/repos/asf/kudu/repo Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/2e1d95cb Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/2e1d95cb Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/2e1d95cb Branch: refs/heads/master Commit: 2e1d95cb23e18541d610444483717810ec671bb6 Parents: 245f0a7 Author: Adar Dembo <a...@cloudera.com> Authored: Fri Aug 24 17:06:13 2018 -0700 Committer: Grant Henke <granthe...@apache.org> Committed: Sat Aug 25 02:33:54 2018 +0000 ---------------------------------------------------------------------- .../src/test/java/org/apache/kudu/client/TestSecurity.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/kudu/blob/2e1d95cb/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java ---------------------------------------------------------------------- diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java index 40e4d7d..0ebcc50 100644 --- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java +++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java @@ -65,7 +65,6 @@ public class TestSecurity { }; private void startCluster(Set<Option> opts) throws IOException { - Preconditions.checkState(miniCluster == null); MiniKuduClusterBuilder mcb = new MiniKuduClusterBuilder(); mcb.enableKerberos(); if (opts.contains(Option.LONG_LEADER_ELECTION)) { @@ -444,4 +443,4 @@ public class TestSecurity { Assert.assertThat(cla.getAppendedText(), CoreMatchers.containsString( "Using caller-provided subject with Kerberos principal test-ad...@krbtest.com.")); } -} \ No newline at end of file +}
