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

Illya Yalovyy commented on HIVE-11791:
--------------------------------------

[~hagleitn] , Could you please review results of my unit tests. In particular:
1. when compactExpr(and(true, null)) it was TRUE before your change. Now it is 
null. 
2. when compactExpr(or(true, null)) it was NULL, and now it returns TRUE.

These changes in behavior totally make sense, I just want to confirm it was 
done intentionally and will not have any side effects.
 
Those changes look strange to me:

when compactExpr(and(true, true)) it was TRUE, now it return NULL.
when compactExpr(or(isNull(col1), false)) it was isNull(col1), now it returns 
NULL 

Also, it would be great if you suggest other test cases for this method.


> Add unit test for HIVE-10122
> ----------------------------
>
>                 Key: HIVE-11791
>                 URL: https://issues.apache.org/jira/browse/HIVE-11791
>             Project: Hive
>          Issue Type: Test
>          Components: Metastore
>    Affects Versions: 1.1.0
>            Reporter: Illya Yalovyy
>            Priority: Minor
>         Attachments: HIVE-11791.patch
>
>
> Unit tests for PartitionPruner.compactExpr()



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to