You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ct...@apache.org on 2016/08/17 00:18:29 UTC

[01/10] accumulo git commit: ACCUMULO-4315 Remove confusing log message

Repository: accumulo
Updated Branches:
  refs/heads/1.6 02ac592cb -> 7efbbd5dd
  refs/heads/1.7 8aaef9e7a -> 7eb4a0962
  refs/heads/1.8 711b4211d -> 5e53631b2
  refs/heads/master 9091d8065 -> 418289cbd


ACCUMULO-4315 Remove confusing log message

Remove an unnecessarily confusing log message, about using a volume
which is not contained in instance.volumes. There's no reason to warn
about this, because it's not a problem. Further, it is expected in some
cases, such as bulk import, when files are imported from outside an
Accumulo "Volume".


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

Branch: refs/heads/1.6
Commit: 7efbbd5dd762debaba39b1cdaa6011a06b5d299a
Parents: 02ac592
Author: Christopher Tubbs <ct...@apache.org>
Authored: Tue Aug 16 20:12:24 2016 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Tue Aug 16 20:12:24 2016 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/server/fs/VolumeManagerImpl.java  | 6 ------
 1 file changed, 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/7efbbd5d/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 70332da..756af43 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
@@ -308,12 +308,6 @@ public class VolumeManagerImpl implements VolumeManager {
               return candidateVolume;
             }
           }
-
-          // For the same reason as we can have multiple Volumes within a single filesystem
-          // we could also not find a matching one. We should still provide a Volume with the
-          // correct FileSystem even though we don't know what the proper base dir is
-          // e.g. Files on volumes that are now removed
-          log.debug("Found candidate Volumes for Path but none of the Volumes are valid for the candidates: " + path);
         } else {
           log.debug("Could not determine volume for Path: " + path);
         }


[08/10] accumulo git commit: Merge branch '1.7' into 1.8

Posted by ct...@apache.org.
Merge branch '1.7' into 1.8


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

Branch: refs/heads/1.8
Commit: 5e53631b2bb5cc9f85de728d7217fe495dfe4fdd
Parents: 711b421 7eb4a09
Author: Christopher Tubbs <ct...@apache.org>
Authored: Tue Aug 16 20:17:48 2016 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Tue Aug 16 20:17:48 2016 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/server/fs/VolumeManagerImpl.java  | 6 ------
 1 file changed, 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/5e53631b/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
----------------------------------------------------------------------


[05/10] accumulo git commit: Merge branch '1.6' into 1.7

Posted by ct...@apache.org.
Merge branch '1.6' into 1.7


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

Branch: refs/heads/1.8
Commit: 7eb4a09625462b9fcff1b0a48b56b10a3096af5d
Parents: 8aaef9e 7efbbd5
Author: Christopher Tubbs <ct...@apache.org>
Authored: Tue Aug 16 20:17:24 2016 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Tue Aug 16 20:17:24 2016 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/server/fs/VolumeManagerImpl.java  | 6 ------
 1 file changed, 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/7eb4a096/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
----------------------------------------------------------------------


[06/10] accumulo git commit: Merge branch '1.6' into 1.7

Posted by ct...@apache.org.
Merge branch '1.6' into 1.7


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

Branch: refs/heads/1.7
Commit: 7eb4a09625462b9fcff1b0a48b56b10a3096af5d
Parents: 8aaef9e 7efbbd5
Author: Christopher Tubbs <ct...@apache.org>
Authored: Tue Aug 16 20:17:24 2016 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Tue Aug 16 20:17:24 2016 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/server/fs/VolumeManagerImpl.java  | 6 ------
 1 file changed, 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/7eb4a096/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
----------------------------------------------------------------------


[03/10] accumulo git commit: ACCUMULO-4315 Remove confusing log message

Posted by ct...@apache.org.
ACCUMULO-4315 Remove confusing log message

Remove an unnecessarily confusing log message, about using a volume
which is not contained in instance.volumes. There's no reason to warn
about this, because it's not a problem. Further, it is expected in some
cases, such as bulk import, when files are imported from outside an
Accumulo "Volume".


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

Branch: refs/heads/1.8
Commit: 7efbbd5dd762debaba39b1cdaa6011a06b5d299a
Parents: 02ac592
Author: Christopher Tubbs <ct...@apache.org>
Authored: Tue Aug 16 20:12:24 2016 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Tue Aug 16 20:12:24 2016 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/server/fs/VolumeManagerImpl.java  | 6 ------
 1 file changed, 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/7efbbd5d/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 70332da..756af43 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
@@ -308,12 +308,6 @@ public class VolumeManagerImpl implements VolumeManager {
               return candidateVolume;
             }
           }
-
-          // For the same reason as we can have multiple Volumes within a single filesystem
-          // we could also not find a matching one. We should still provide a Volume with the
-          // correct FileSystem even though we don't know what the proper base dir is
-          // e.g. Files on volumes that are now removed
-          log.debug("Found candidate Volumes for Path but none of the Volumes are valid for the candidates: " + path);
         } else {
           log.debug("Could not determine volume for Path: " + path);
         }


[09/10] accumulo git commit: Merge branch '1.7' into 1.8

Posted by ct...@apache.org.
Merge branch '1.7' into 1.8


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

Branch: refs/heads/master
Commit: 5e53631b2bb5cc9f85de728d7217fe495dfe4fdd
Parents: 711b421 7eb4a09
Author: Christopher Tubbs <ct...@apache.org>
Authored: Tue Aug 16 20:17:48 2016 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Tue Aug 16 20:17:48 2016 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/server/fs/VolumeManagerImpl.java  | 6 ------
 1 file changed, 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/5e53631b/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
----------------------------------------------------------------------


[04/10] accumulo git commit: ACCUMULO-4315 Remove confusing log message

Posted by ct...@apache.org.
ACCUMULO-4315 Remove confusing log message

Remove an unnecessarily confusing log message, about using a volume
which is not contained in instance.volumes. There's no reason to warn
about this, because it's not a problem. Further, it is expected in some
cases, such as bulk import, when files are imported from outside an
Accumulo "Volume".


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

Branch: refs/heads/master
Commit: 7efbbd5dd762debaba39b1cdaa6011a06b5d299a
Parents: 02ac592
Author: Christopher Tubbs <ct...@apache.org>
Authored: Tue Aug 16 20:12:24 2016 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Tue Aug 16 20:12:24 2016 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/server/fs/VolumeManagerImpl.java  | 6 ------
 1 file changed, 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/7efbbd5d/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 70332da..756af43 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
@@ -308,12 +308,6 @@ public class VolumeManagerImpl implements VolumeManager {
               return candidateVolume;
             }
           }
-
-          // For the same reason as we can have multiple Volumes within a single filesystem
-          // we could also not find a matching one. We should still provide a Volume with the
-          // correct FileSystem even though we don't know what the proper base dir is
-          // e.g. Files on volumes that are now removed
-          log.debug("Found candidate Volumes for Path but none of the Volumes are valid for the candidates: " + path);
         } else {
           log.debug("Could not determine volume for Path: " + path);
         }


[02/10] accumulo git commit: ACCUMULO-4315 Remove confusing log message

Posted by ct...@apache.org.
ACCUMULO-4315 Remove confusing log message

Remove an unnecessarily confusing log message, about using a volume
which is not contained in instance.volumes. There's no reason to warn
about this, because it's not a problem. Further, it is expected in some
cases, such as bulk import, when files are imported from outside an
Accumulo "Volume".


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

Branch: refs/heads/1.7
Commit: 7efbbd5dd762debaba39b1cdaa6011a06b5d299a
Parents: 02ac592
Author: Christopher Tubbs <ct...@apache.org>
Authored: Tue Aug 16 20:12:24 2016 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Tue Aug 16 20:12:24 2016 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/server/fs/VolumeManagerImpl.java  | 6 ------
 1 file changed, 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/7efbbd5d/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 70332da..756af43 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
@@ -308,12 +308,6 @@ public class VolumeManagerImpl implements VolumeManager {
               return candidateVolume;
             }
           }
-
-          // For the same reason as we can have multiple Volumes within a single filesystem
-          // we could also not find a matching one. We should still provide a Volume with the
-          // correct FileSystem even though we don't know what the proper base dir is
-          // e.g. Files on volumes that are now removed
-          log.debug("Found candidate Volumes for Path but none of the Volumes are valid for the candidates: " + path);
         } else {
           log.debug("Could not determine volume for Path: " + path);
         }


[10/10] accumulo git commit: Merge branch '1.8'

Posted by ct...@apache.org.
Merge branch '1.8'


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

Branch: refs/heads/master
Commit: 418289cbdd8ba619eabe50344354d166e003e521
Parents: 9091d80 5e53631
Author: Christopher Tubbs <ct...@apache.org>
Authored: Tue Aug 16 20:18:01 2016 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Tue Aug 16 20:18:01 2016 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/server/fs/VolumeManagerImpl.java  | 6 ------
 1 file changed, 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/418289cb/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
----------------------------------------------------------------------


[07/10] accumulo git commit: Merge branch '1.6' into 1.7

Posted by ct...@apache.org.
Merge branch '1.6' into 1.7


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

Branch: refs/heads/master
Commit: 7eb4a09625462b9fcff1b0a48b56b10a3096af5d
Parents: 8aaef9e 7efbbd5
Author: Christopher Tubbs <ct...@apache.org>
Authored: Tue Aug 16 20:17:24 2016 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Tue Aug 16 20:17:24 2016 -0400

----------------------------------------------------------------------
 .../java/org/apache/accumulo/server/fs/VolumeManagerImpl.java  | 6 ------
 1 file changed, 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/7eb4a096/server/base/src/main/java/org/apache/accumulo/server/fs/VolumeManagerImpl.java
----------------------------------------------------------------------