You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pig.apache.org by "Alan Gates (JIRA)" <ji...@apache.org> on 2013/02/04 18:42:13 UTC

[jira] [Updated] (PIG-2834) MultiStorage requires unused constructor argument

     [ 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