You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by el...@apache.org on 2015/06/06 07:09:10 UTC
accumulo git commit: ACCUMULO-3871 Fix findbugs failures
Repository: accumulo
Updated Branches:
refs/heads/master effc692dd -> 1ae4655d2
ACCUMULO-3871 Fix findbugs failures
Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/1ae4655d
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/1ae4655d
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/1ae4655d
Branch: refs/heads/master
Commit: 1ae4655d26912cf60cbfc46c0ab8de3556da32a5
Parents: effc692
Author: Josh Elser <el...@apache.org>
Authored: Sat Jun 6 01:08:47 2015 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Sat Jun 6 01:08:47 2015 -0400
----------------------------------------------------------------------
test/src/main/java/org/apache/accumulo/harness/TestingKdc.java | 3 ++-
.../java/org/apache/accumulo/test/UserCompactionStrategyIT.java | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/accumulo/blob/1ae4655d/test/src/main/java/org/apache/accumulo/harness/TestingKdc.java
----------------------------------------------------------------------
diff --git a/test/src/main/java/org/apache/accumulo/harness/TestingKdc.java b/test/src/main/java/org/apache/accumulo/harness/TestingKdc.java
index e05b9b0..1f84fc8 100644
--- a/test/src/main/java/org/apache/accumulo/harness/TestingKdc.java
+++ b/test/src/main/java/org/apache/accumulo/harness/TestingKdc.java
@@ -56,7 +56,8 @@ public class TestingKdc {
private static File computeKdcDir() {
File targetDir = new File(System.getProperty("user.dir"), "target");
- targetDir.mkdirs();
+ if (!targetDir.exists())
+ Assert.assertTrue(targetDir.mkdirs());
Assert.assertTrue("Could not find Maven target directory: " + targetDir, targetDir.exists() && targetDir.isDirectory());
// Create the directories: target/kerberos/minikdc
http://git-wip-us.apache.org/repos/asf/accumulo/blob/1ae4655d/test/src/main/java/org/apache/accumulo/test/UserCompactionStrategyIT.java
----------------------------------------------------------------------
diff --git a/test/src/main/java/org/apache/accumulo/test/UserCompactionStrategyIT.java b/test/src/main/java/org/apache/accumulo/test/UserCompactionStrategyIT.java
index 127f779..9c319d2 100644
--- a/test/src/main/java/org/apache/accumulo/test/UserCompactionStrategyIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/UserCompactionStrategyIT.java
@@ -133,7 +133,7 @@ public class UserCompactionStrategyIT extends AccumuloClusterHarness {
final Connector c = getConnector();
final String tableName = getUniqueNames(1)[0];
File target = new File(System.getProperty("user.dir"), "target");
- target.mkdirs();
+ Assert.assertTrue(target.mkdirs());
Assert.assertTrue(target.exists() && target.isDirectory());
File destFile = installJar(target, "/TestCompactionStrat.jar");
c.tableOperations().create(tableName);