You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by gr...@apache.org on 2018/08/25 03:03:17 UTC
[2/3] kudu git commit: TestSecurity: remove errant Precondition
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 <da...@apache.org>
Tested-by: Kudu Jenkins
Reviewed-by: Grant Henke <gr...@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 <ad...@cloudera.com>
Authored: Fri Aug 24 17:06:13 2018 -0700
Committer: Grant Henke <gr...@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-admin@KRBTEST.COM."));
}
-}
\ No newline at end of file
+}