alamb commented on code in PR #5322: URL: https://github.com/apache/arrow-datafusion/pull/5322#discussion_r1115701082
########## datafusion/physical-expr/src/intervals/mod.rs: ########## @@ -0,0 +1,26 @@ +// Licensed to the Apache Software Foundation (ASF) under one Review Comment: > At the moment, the current implementations are somewhat limited in their ability to handle complex expressions, and it is challenging to expand them beyond their current capabilities. However, with the maturation of the interval arithmetics library, we can explore more advanced expression analysis, which holds the potential for significant progress. I agree about the current approaches being limited. It would be great to have the general purpose interval analysis library in this PR underpinning all three uses. It would be great to do a proof of concept (not in this PR) of using the interval library in place of the existing pruning or range analysis. When this PR is merged, I'll plan to file a follow on ticket. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
