You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2022/02/23 05:48:02 UTC

[GitHub] [calcite] NobiGo commented on pull request #2727: [CALCITE-1794] Simplify AND numeric comparisons when overlapping

NobiGo commented on pull request #2727:
URL: https://github.com/apache/calcite/pull/2727#issuecomment-1048467001


   > Hi @NobiGo, in the associated Jira ticket you suggested a re-ordering of predicates, from some test changes it seems that this is indeed happening now, but I see no code change related to that. Is the addition of the case for `CAST` causing that somehow?
   
   @asolimando Yes. It looks need to reorder it, but as @julianhyde is concerned (It Will introduce plan diffs and make this more expensive). So When I try to find another way to resolve it, I found after the RexNode is optimized, The RexNode is converted to a Search argument, Then this case can be simplified to a point.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@calcite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org