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

Gopal V commented on HIVE-11398:
--------------------------------

Patch looks good, there are several golden files to be updated to remove the 
stray parenthesis.

There are some other failures though to look closer at

TestCliDriver.testCliDriver_pcr
TestSparkCliDriver.testCliDriver_pcr
TestSparkCliDriver.testCliDriver_input_part2
TestCliDriver.testCliDriver_input_part2

and udf_or.q.out seems to be just a comment issue?

> Parse wide OR and wide AND trees to flat OR/AND trees
> -----------------------------------------------------
>
>                 Key: HIVE-11398
>                 URL: https://issues.apache.org/jira/browse/HIVE-11398
>             Project: Hive
>          Issue Type: New Feature
>          Components: Logical Optimizer, UDF
>    Affects Versions: 1.3.0, 2.0.0
>            Reporter: Gopal V
>            Assignee: Jesus Camacho Rodriguez
>         Attachments: HIVE-11398.patch
>
>
> Deep trees of AND/OR are hard to traverse particularly when they are merely 
> the same structure in nested form as a version of the operator that takes an 
> arbitrary number of args.
> One potential way to convert the DFS searches into a simpler BFS search is to 
> introduce a new Operator pair named ALL and ANY.
> ALL(A, B, C, D, E) represents AND(AND(AND(AND(E, D), C), B), A)
> ANY(A, B, C, D, E) represents OR(OR(OR(OR(E, D), C),B),A)
> The SemanticAnalyser would be responsible for generating these operators and 
> this would mean that the depth and complexity of traversals for the simplest 
> case of wide AND/OR trees would be trivial.



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

Reply via email to