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 495851033C for ; Mon, 3 Feb 2014 18:18:11 +0000 (UTC) Received: (qmail 15106 invoked by uid 500); 3 Feb 2014 18:18:10 -0000 Delivered-To: apmail-accumulo-notifications-archive@accumulo.apache.org Received: (qmail 15074 invoked by uid 500); 3 Feb 2014 18:18:09 -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 15064 invoked by uid 99); 3 Feb 2014 18:18:09 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 03 Feb 2014 18:18:09 +0000 Date: Mon, 3 Feb 2014 18:18:09 +0000 (UTC) From: "Josh Elser (JIRA)" To: notifications@accumulo.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Issue Comment Deleted] (ACCUMULO-2297) Additional unnecessary member on InputFormatBase 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-2297?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Josh Elser updated ACCUMULO-2297: --------------------------------- Comment: was deleted (was: Granted it would be bad practice, but if a user decided to put their input format in the same package as the Accumulo one, they could have access to it and suddenly their code would break.) > Additional unnecessary member on InputFormatBase > ------------------------------------------------ > > Key: ACCUMULO-2297 > URL: https://issues.apache.org/jira/browse/ACCUMULO-2297 > Project: Accumulo > Issue Type: Bug > Components: client > Affects Versions: 1.4.4, 1.5.0 > Reporter: Josh Elser > Assignee: Josh Elser > Priority: Minor > Fix For: 1.4.5, 1.5.1, 1.6.0 > > > Noticed in 1.4.5 and 1.5.1 that there's an extra currentValue member defined in the RecordReaderBase that is never actually used (there's a currentV member that's used instead), conversely to the currentK and the currentKey members which are returned to the client and used for progress reporting, respectively. > We probably shouldn't change these for the current versions (as we could break compilation across minor releases), but it likely makes sense to remove this extra, unnecessary variable before 1.6 goes out. -- This message was sent by Atlassian JIRA (v6.1.5#6160)