accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ctubb...@apache.org
Subject [01/10] accumulo git commit: ACCUMULO-4315 Remove confusing log message
Date Wed, 17 Aug 2016 00:18:29 GMT
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 <ctubbsii@apache.org>
Authored: Tue Aug 16 20:12:24 2016 -0400
Committer: Christopher Tubbs <ctubbsii@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);
         }


Mime
View raw message