Return-Path: X-Original-To: apmail-accumulo-notifications-archive@minotaur.apache.org Delivered-To: apmail-accumulo-notifications-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 539FD11966 for ; Mon, 16 Jun 2014 14:34:41 +0000 (UTC) Received: (qmail 84310 invoked by uid 500); 16 Jun 2014 14:34:41 -0000 Delivered-To: apmail-accumulo-notifications-archive@accumulo.apache.org Received: (qmail 84268 invoked by uid 500); 16 Jun 2014 14:34:41 -0000 Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: jira@apache.org Delivered-To: mailing list notifications@accumulo.apache.org Received: (qmail 82511 invoked by uid 99); 16 Jun 2014 12:48:01 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 16 Jun 2014 12:48:01 +0000 Date: Mon, 16 Jun 2014 12:48:01 +0000 (UTC) From: "Eric Newton (JIRA)" To: notifications@accumulo.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (ACCUMULO-2877) CachedConfiguration.setInstance never used outside class MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/ACCUMULO-2877?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14032397#comment-14032397 ] Eric Newton commented on ACCUMULO-2877: --------------------------------------- I know of at least one user that calls {{setInstance}} because they do not load their hadoop information using the classpath. They are using the AccumuloOutputFormat and write to HDFS. There's no way for them to specify their filesystem since the client code assumes the ability to load a Configuration from the classpath. Calling {{setInstance}} allows them to provide the configuration through external means. Please do not remove this method. > CachedConfiguration.setInstance never used outside class > -------------------------------------------------------- > > Key: ACCUMULO-2877 > URL: https://issues.apache.org/jira/browse/ACCUMULO-2877 > Project: Accumulo > Issue Type: Improvement > Affects Versions: 1.5.1, 1.6.0 > Reporter: Bill Havanki > Assignee: Anson Liu > Priority: Trivial > Labels: newbie, summit2014 > Fix For: 1.5.2 > > Attachments: ACCUMULO-2877.patch, ACCUMULO-2877.patch > > > The method {{CachedConfiguration.setInstance()}} is a public static method but is never used outside the class. It should be hidden or removed. -- This message was sent by Atlassian JIRA (v6.2#6252)