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 <ctubb...@apache.org> Authored: Tue Aug 16 20:12:24 2016 -0400 Committer: Christopher Tubbs <ctubb...@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); }
