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

Rohini Palaniswamy commented on PIG-4002:
-----------------------------------------

Tez has combiner working on both map and reduce. We should only turn it off on 
the map side in Tez. This patch I see only addresses MR. We need to address Tez 
as well or open a separate jira for that. 

> Disable combiner when map-side aggregation is used
> --------------------------------------------------
>
>                 Key: PIG-4002
>                 URL: https://issues.apache.org/jira/browse/PIG-4002
>             Project: Pig
>          Issue Type: Improvement
>    Affects Versions: 0.13.0
>            Reporter: Travis Woodruff
>            Assignee: Travis Woodruff
>            Priority: Minor
>         Attachments: PIG-4002-1.patch
>
>
> This may be controversial, so I'd like others' opinions on this.
> It is not currently possible to disable the combiner and use map-side 
> aggregation at the same time. This is a problematic since map-side 
> aggregation effectively combines in the mapper, so running the combiner adds 
> expensive combiner execution (combiner requires deserialization & 
> reserialization) for little to no value.
> PIG-2829 had a patch to disable the combiner when map-side aggregation is 
> used (along with some other changes). This was never integrated because the 
> map-side aggregation code was redone while this was in progress.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to