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 7C15C1117B for ; Tue, 12 Aug 2014 19:56:14 +0000 (UTC) Received: (qmail 13703 invoked by uid 500); 12 Aug 2014 19:56:13 -0000 Delivered-To: apmail-accumulo-notifications-archive@accumulo.apache.org Received: (qmail 13655 invoked by uid 500); 12 Aug 2014 19:56:13 -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 13434 invoked by uid 99); 12 Aug 2014 19:56:13 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 12 Aug 2014 19:56:13 +0000 Date: Tue, 12 Aug 2014 19:56:13 +0000 (UTC) From: "Christopher Tubbs (JIRA)" To: notifications@accumulo.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (ACCUMULO-3019) Fix ServerConfiguration deprecation 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-3019?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14094580#comment-14094580 ] Christopher Tubbs commented on ACCUMULO-3019: --------------------------------------------- Discussed with [~ecn] and it might make the most sense to simply remove the deprecation on the 1.6 branch, to get rid of the noise. The migration to the ServerConfigurationFactory is already in motion in the master branch, so I think that is the best route. > Fix ServerConfiguration deprecation > ----------------------------------- > > Key: ACCUMULO-3019 > URL: https://issues.apache.org/jira/browse/ACCUMULO-3019 > Project: Accumulo > Issue Type: Sub-task > Components: master, tserver > Reporter: Eric Newton > Assignee: Eric Newton > Fix For: 1.6.1, 1.7.0 > > Time Spent: 0.5h > Remaining Estimate: 0h > > The deprecation of ServerConfiguration causes two problems: > 1) eclipse is showing me ~300 warnings, so I'm missing other warnings in the noise > 2) ServerConfiguration is passed to some plug-ins (Balancer, at least), so any existing plug-ins are broken by the change. > In the past, I would have been fine with just switching to the new class {{ServerConfigurationFactory}} but a growing user base and petabytes of existing systems gives me pause. > Instead, I'm thinking we hoist the most popular methods to an abstract ServerConfiguration, and provide ServerConfiguationFactory as the implementation. This way the extensions continue to work. > I'm still not sure what to do about static calls to get the SiteConfiguration. -- This message was sent by Atlassian JIRA (v6.2#6252)