wjones127 commented on issue #6171:
URL: 
https://github.com/apache/arrow-datafusion/issues/6171#issuecomment-1704411059

   > It would be amazing to incorporate bounds analysis into expr 
simplification, though I would like to request we don't use yet another 
representation of ranges / bounds
   
   I'm taking a look at this right now. Two issues I see right now:
   
   1. The `Interval` bounds doesn't include any information about nullability. 
I'd like to simplify expressions like `X IS NOT NULL` to `true` or `false` if 
null statistics support that simplification.
   2. The interval library operates on physical expressions, while the 
simplification operates on logical expressions.
   
   I'm working on a PR right now that will include a struct that is parallel to 
`Interval` for logical expressions that includes the null information. Once I 
have that working I'll see if it's worth consolidating that with the existing 
`Interval` struct.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to