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 C52E61031C for ; Wed, 19 Mar 2014 14:16:47 +0000 (UTC) Received: (qmail 86572 invoked by uid 500); 19 Mar 2014 14:16:45 -0000 Delivered-To: apmail-accumulo-notifications-archive@accumulo.apache.org Received: (qmail 86532 invoked by uid 500); 19 Mar 2014 14:16:44 -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 86522 invoked by uid 99); 19 Mar 2014 14:16:44 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 19 Mar 2014 14:16:44 +0000 Date: Wed, 19 Mar 2014 14:16:43 +0000 (UTC) From: "Bill Havanki (JIRA)" To: notifications@accumulo.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (ACCUMULO-2497) InstanceFactory 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-2497?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Bill Havanki updated ACCUMULO-2497: ----------------------------------- Description: Working on ACCUMULO-2470 (unit tests for server/base) is made difficult by the heavy dependence on static method calls in our architecture. This ticket is for introducing an {{InstanceFactory}} interface that will greatly ease testing of code that currently retrieves instances via static methods (e.g., {{HdfsZooInstance.getInstance()}}). Limitations: * Work here should be limited to creating {{InstanceFactory}} and its implementations, and not integrating it into existing code (yet). * Work is limited to 1.6 and later. To avoid perturbing the 1.6.0 release no new code may be introduced into the public API ({{Instance}} is in the public API). * A refactoring of the client API is being considered (see ACCUMULO-2128), so this change should not further complicate - and should hopefully help - that future effort. was: Working on ACCUMULO-2470 (unit tests for server/base) is made difficult by the heavy dependence on static method calls in our architecture. This ticket is for introducing an {{InstanceFactory}} interface that will greatly ease testing of code that currently retrieves instances via static methods (e.g., {{HdfsZooInstance.getInstance()}}. Limitations: * Work here should be limited to creating {{InstanceFactory}} and its implementations, and not integrating it into existing code (yet). * Work is limited to 1.6 and later. To avoid perturbing the 1.6.0 release no new code may be introduced into the public API ({{Instance}} is in the public API). * A refactoring of the client API is being considered (see ACCUMULO-2128), so this change should not further complicate - and should hopefully help - that future effort. > InstanceFactory > --------------- > > Key: ACCUMULO-2497 > URL: https://issues.apache.org/jira/browse/ACCUMULO-2497 > Project: Accumulo > Issue Type: Improvement > Components: client > Reporter: Bill Havanki > Assignee: Bill Havanki > Priority: Minor > Labels: factory, static, testability > > Working on ACCUMULO-2470 (unit tests for server/base) is made difficult by the heavy dependence on static method calls in our architecture. This ticket is for introducing an {{InstanceFactory}} interface that will greatly ease testing of code that currently retrieves instances via static methods (e.g., {{HdfsZooInstance.getInstance()}}). > Limitations: > * Work here should be limited to creating {{InstanceFactory}} and its implementations, and not integrating it into existing code (yet). > * Work is limited to 1.6 and later. To avoid perturbing the 1.6.0 release no new code may be introduced into the public API ({{Instance}} is in the public API). > * A refactoring of the client API is being considered (see ACCUMULO-2128), so this change should not further complicate - and should hopefully help - that future effort. -- This message was sent by Atlassian JIRA (v6.2#6252)