Return-Path: X-Original-To: apmail-pig-dev-archive@www.apache.org Delivered-To: apmail-pig-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 3E81AED66 for ; Mon, 4 Feb 2013 17:42:14 +0000 (UTC) Received: (qmail 92533 invoked by uid 500); 4 Feb 2013 17:42:13 -0000 Delivered-To: apmail-pig-dev-archive@pig.apache.org Received: (qmail 92478 invoked by uid 500); 4 Feb 2013 17:42:13 -0000 Mailing-List: contact dev-help@pig.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@pig.apache.org Delivered-To: mailing list dev@pig.apache.org Received: (qmail 92440 invoked by uid 500); 4 Feb 2013 17:42:13 -0000 Delivered-To: apmail-hadoop-pig-dev@hadoop.apache.org Received: (qmail 92427 invoked by uid 99); 4 Feb 2013 17:42:13 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 04 Feb 2013 17:42:13 +0000 Date: Mon, 4 Feb 2013 17:42:13 +0000 (UTC) From: "Alan Gates (JIRA)" To: pig-dev@hadoop.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (PIG-2834) MultiStorage requires unused constructor argument 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/PIG-2834?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Alan Gates updated PIG-2834: ---------------------------- Status: Open (was: Patch Available) These changes break backward compatibility for users of MultiStorage. I agree the parentPathStr is unused and not required, but you need to deprecate the existing contructors without removing them and add new ones that don't take parentPathStr. This allows current users a path forward without breaking their code. > MultiStorage requires unused constructor argument > ------------------------------------------------- > > Key: PIG-2834 > URL: https://issues.apache.org/jira/browse/PIG-2834 > Project: Pig > Issue Type: Improvement > Components: data > Affects Versions: 0.10.0, 0.11 > Environment: Linux > Reporter: Danny Antonetti > Priority: Trivial > Labels: newbie > Fix For: 0.12 > > Attachments: MultiStorage.patch > > > each constructor in > org.apache.pig.piggybank.storage.MultiStorage > requires a constructor argument 'parentPathStr", that has no meaningful usage. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira