You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ec...@apache.org on 2014/08/07 21:34:06 UTC

git commit: ACCUMULO-3006 add messages to {{IllegalArgumentException}}s

Repository: accumulo
Updated Branches:
  refs/heads/1.6.1-SNAPSHOT d34887a23 -> 1b459a020


ACCUMULO-3006 add messages to {{IllegalArgumentException}}s


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/1b459a02
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/1b459a02
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/1b459a02

Branch: refs/heads/1.6.1-SNAPSHOT
Commit: 1b459a0205501389c5c6a2f3172030edeae34779
Parents: d34887a
Author: Eric C. Newton <er...@gmail.com>
Authored: Thu Aug 7 15:33:58 2014 -0400
Committer: Eric C. Newton <er...@gmail.com>
Committed: Thu Aug 7 15:33:58 2014 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/server/fs/VolumeManagerImpl.java   | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/1b459a02/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java b/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
index 877b9a6..8ddeb4f 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
@@ -397,11 +397,10 @@ public class VolumeManagerImpl implements VolumeManager {
     // The "default" Volume for Accumulo (in case no volumes are specified)
     for (String volumeUriOrDir : VolumeConfiguration.getVolumeUris(conf)) {
       if (volumeUriOrDir.equals(DEFAULT))
-        // Cannot re-define the default volume
-        throw new IllegalArgumentException();
+        throw new IllegalArgumentException("Cannot re-define the default volume");
 
       if (volumeUriOrDir.startsWith("viewfs"))
-        throw new IllegalArgumentException();
+        throw new IllegalArgumentException("Cannot use viewfs as a volume");
 
       // We require a URI here, fail if it doesn't look like one
       if (volumeUriOrDir.contains(":")) {