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

Brock Noland commented on HIVE-5556:
------------------------------------

I have no issue with a q file test in addition to unit tests but the method you 
created applyEqualityPredicateToQBJoinTree is a perfect candidate for unit 
testing. WIthout a ridiculous number of queries there is no way we can say that 
method is well tested. I looked at the code it's replacing a long while back 
and thought "Oh man this should really be unit tested" which is why I commented 
here in the first place.

> Pushdown join conditions
> ------------------------
>
>                 Key: HIVE-5556
>                 URL: https://issues.apache.org/jira/browse/HIVE-5556
>             Project: Hive
>          Issue Type: Sub-task
>          Components: Query Processor
>            Reporter: Harish Butani
>            Assignee: Harish Butani
>         Attachments: HIVE-5556.1.patch
>
>
> See details in HIVE-5555



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to