wjones127 commented on issue #6171: URL: https://github.com/apache/arrow-datafusion/issues/6171#issuecomment-1705896888
> I will also think about how we can add nullability support to the interval library without resulting in large changes or performance impacts. Moving it outside of physical-expr as a general purpose library is fairly trivial. Could I extract the `Interval` struct into `datafusion_common`, add the nullability field, and then leave a note in the `cp_solver` module that nullability isn't handled yet? Here is the null tracking definition I created: https://github.com/apache/arrow-datafusion/blob/a6b57e38eb00da2a6c5396dca0b5f1772578ac78/datafusion/optimizer/src/simplify_expressions/guarantees.rs#L71-L84 Does that seem good to you? -- 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]
