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

Ashutosh Chauhan commented on HIVE-5186:
----------------------------------------

I am not sure we want this. Wherever possible RSDeDup should compact the query 
plan (since it avoids adding new operators in piepline). Only cases which can 
not be handled by RSDeDup, Correlation Optimizer should kick in. 

In this patch, it seems like you are removing a case which can be handled by 
RSDeDup.
                
> Remove JoinReducerProc from ReduceSinkDeDuplication
> ---------------------------------------------------
>
>                 Key: HIVE-5186
>                 URL: https://issues.apache.org/jira/browse/HIVE-5186
>             Project: Hive
>          Issue Type: Improvement
>            Reporter: Yin Huai
>            Assignee: Yin Huai
>            Priority: Minor
>         Attachments: HIVE-5186.1.patch.txt
>
>
> Correlation Optimizer will take care patterns involving JoinOperator. We can 
> remove JoinReducerProc from ReduceSinkDeDuplication.

--
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

Reply via email to