Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id B4EDB200B71 for ; Wed, 17 Aug 2016 02:18:30 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id B3B4A160AC4; Wed, 17 Aug 2016 00:18:30 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 110FA160AA8 for ; Wed, 17 Aug 2016 02:18:29 +0200 (CEST) Received: (qmail 38301 invoked by uid 500); 17 Aug 2016 00:18:29 -0000 Mailing-List: contact commits-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@accumulo.apache.org Delivered-To: mailing list commits@accumulo.apache.org Received: (qmail 38284 invoked by uid 99); 17 Aug 2016 00:18:29 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 17 Aug 2016 00:18:29 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 2693CDFF8D; Wed, 17 Aug 2016 00:18:29 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: ctubbsii@apache.org To: commits@accumulo.apache.org Date: Wed, 17 Aug 2016 00:18:29 -0000 Message-Id: <28419dd297844777a8fa936c785895a8@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [01/10] accumulo git commit: ACCUMULO-4315 Remove confusing log message archived-at: Wed, 17 Aug 2016 00:18:30 -0000 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 Authored: Tue Aug 16 20:12:24 2016 -0400 Committer: Christopher Tubbs 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); }