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

Aleksey Plekhanov commented on CALCITE-5288:
--------------------------------------------

[~julianhyde], thanks, I've changed the subject and example.

What about proposed fix? (I've prepared and attach the pull request with 
described changes)

> Expression '(a > 5 and a < 15) or (a > 10 and a < 20)' should be simplified 
> to 'SEARCH(a, Sarg[(5..20)])'
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: CALCITE-5288
>                 URL: https://issues.apache.org/jira/browse/CALCITE-5288
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: Aleksey Plekhanov
>            Assignee: Aleksey Plekhanov
>            Priority: Major
>
> If there is only one range in search/sarg after simplification, the result of 
> sarg collector is ignored ({{{}SargCollector#needToFix(){}}} method).
> Expression  {{(a > 5 and a < 15) or (a > 10 and a < 20)}} is simplified to 
> {{OR(SEARCH(a, Sarg[(5..15)]), SEARCH(a, Sarg[(10..20)]))}} but expected 
> {{{}SEARCH(a, Sarg[(5..20)]){}}}.
> If there are more the one intervals or if open interval exists, expressions 
> simplified as expected.
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to