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

Gopal V commented on HIVE-17399:
--------------------------------

LGTM - +1 tests pending.

One minor nit - "assert false;" should be followed by a return, since not 
everyone runs with -ea.



> Do not remove semijoin branch if it feeds to TS->DPP_EVENT
> ----------------------------------------------------------
>
>                 Key: HIVE-17399
>                 URL: https://issues.apache.org/jira/browse/HIVE-17399
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Deepak Jaiswal
>            Assignee: Deepak Jaiswal
>         Attachments: HIVE-17399.1.patch, HIVE-17399.2.patch
>
>
> If there is an incoming semijoin branch to a TS which has DPP event, then try 
> to keep it as it may serve as an excellent filter for DPP thus reducing the 
> input to join drastically.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to