pepijnve commented on PR #17813: URL: https://github.com/apache/datafusion/pull/17813#issuecomment-3411675110
@comphead I should probably have been a bit more specific. The question I have for you relates to #13081. In that PR, a change was added at https://github.com/apache/datafusion/blob/057583d5c02fb6b555bdd64b87cf596daa05d70f/datafusion/optimizer/src/utils.rs#L135 where a logical expression is being evaluated using default `ExecutionProps`. In this PR I have a similar need to const evaluate a logical expression without having `ExecutionProps` available. I was a bit concerned to do so using default values because you're missing some possibly essential information. I was wondering if you had any insight into this. Is the change in #13081 possibly wrong because of the default props? Or am I worrying about a non-issue? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
