[jira] [Commented] (PIG-3741) Utils.setTmpFileCompressionOnConf can cause side effect for SequenceFileInterStorage

["Julien Le Dem (JIRA)" <ji...@apache.org>]

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

Julien Le Dem commented on PIG-3741:
------------------------------------

Ideally each store would get its own config object, but that would be a major refactoring.
In the meantime, this looks like a good improvement to me.
+1

> Utils.setTmpFileCompressionOnConf can cause side effect for SequenceFileInterStorage
> ------------------------------------------------------------------------------------
>
>                 Key: PIG-3741
>                 URL: https://issues.apache.org/jira/browse/PIG-3741
>             Project: Pig
>          Issue Type: Bug
>            Reporter: Aniket Mokashi
>            Assignee: Aniket Mokashi
>             Fix For: 0.12.1
>
>         Attachments: PIG-3741.patch
>
>
> Currently, Utils.setTmpFileCompressionOnConf(pigContext, conf); is invoked for every job. In case of Seqfile, this api sets mapreduce params on conf to assist SequenceFileInterStorage. However, as a side effect, this might change the behavior of other storers due to these mapred properties. This api should only be called for jobs with intermediate storage.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)