[ 
https://issues.apache.org/jira/browse/PIG-2578?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13439103#comment-13439103
 ] 

Daniel Dai commented on PIG-2578:
---------------------------------

I am fine with reverting the patch. The underlying problem is setStoreLocation 
is the only hook for StoreFunc for multiple purpose. In the javadoc, we shall 
make it clear:
1. Need to distinguish frontend/backend (using UDFContext.isFrontend()), user 
can setup global configuration in the frontend, but can only setup store only 
configuration in the backend
2. When setting up global configuration, need to bear in mind there could be 
multiple store, so config entries can overwrite each other.
                
> Multiple Store-commands mess up mapred.output.dir.
> --------------------------------------------------
>
>                 Key: PIG-2578
>                 URL: https://issues.apache.org/jira/browse/PIG-2578
>             Project: Pig
>          Issue Type: Bug
>    Affects Versions: 0.8.1, 0.9.2
>            Reporter: Mithun Radhakrishnan
>            Assignee: Daniel Dai
>             Fix For: 0.10.0, 0.11
>
>         Attachments: PIG-2578-1.patch
>
>
> When one runs a pig-script with multiple storers, one sees the following:
> 1. When run as a script, Pig launches a single job.
> 2. PigOutputCommitter::setupJob() calls the 
> underlyingOutputCommitter::setupJob(), once for each storer. But the 
> mapred.output.dir is the same for both calls, even though the storers write 
> to different locations. 
> This was originally seen in HCATALOG-276, when HCatalog's end-to-end tests 
> are run against Pig.
> (https://issues.apache.org/jira/browse/HCATALOG-276)
> Sample pig-script (near identical to HCatalog's Pig_Checkin_4 test):
> a = load 'keyvals' using org.apache.hcatalog.pig.HCatLoader();
> split a into b if key<200, c if key >=200;
> store b into 'keyvals_lt200' using org.apache.hcatalog.pig.HCatStorer();
> store c into 'keyvals_ge200' using org.apache.hcatalog.pig.HCatStorer();
> I've suggested a workaround in HCat for the time being, but I think this 
> might be something that needs fixing in Pig.
> Thanks.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to