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 28682108CE for ; Thu, 18 Dec 2014 03:31:16 +0000 (UTC) Received: (qmail 31795 invoked by uid 500); 18 Dec 2014 03:31:16 -0000 Delivered-To: apmail-accumulo-notifications-archive@accumulo.apache.org Received: (qmail 31431 invoked by uid 500); 18 Dec 2014 03:31:15 -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 31416 invoked by uid 99); 18 Dec 2014 03:31:15 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 18 Dec 2014 03:31:15 +0000 Date: Thu, 18 Dec 2014 03:31:15 +0000 (UTC) From: "Corey J. Nolet (JIRA)" To: notifications@accumulo.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (ACCUMULO-3001) Create public getters that represent "current" expected API calls/objects for InputFormat and OutputFormat 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-3001?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Corey J. Nolet updated ACCUMULO-3001: ------------------------------------- Fix Version/s: (was: 1.6.2) 1.6.3 > Create public getters that represent "current" expected API calls/objects for InputFormat and OutputFormat > ---------------------------------------------------------------------------------------------------------- > > Key: ACCUMULO-3001 > URL: https://issues.apache.org/jira/browse/ACCUMULO-3001 > Project: Accumulo > Issue Type: Improvement > Components: client > Reporter: Josh Elser > Fix For: 1.5.3, 1.7.0, 1.6.3 > > > Discussion: http://mail-archives.apache.org/mod_mbox/accumulo-dev/201407.mbox/%3C53C72A29.1060701%40gmail.com%3E > All of the getters on AccumuloInputFormat and AccumuloOutputFormat are protected which prevent clients from accessing them. This is not ideal as it prevents users from introspecting the Configuration to know what values are set within the Configuration. Because elements like the concrete Input/OutputFormat class name is included in the Configuration key, this is very obtuse to try to debug. > The reason this was done is that we often support multiple types of setters which accept a variety of method arguments for different reasons (backwards compat, usually). > I recommend that we provide a single getter to allow users to better debug their mapreduce code. Most likely, the object returned would be the object that is currently reccomended to be used in the public API (e.g. a PasswordToken would be returned even when a user provided a plain String password). -- This message was sent by Atlassian JIRA (v6.3.4#6332)