AdamGS opened a new issue, #20134: URL: https://github.com/apache/datafusion/issues/20134
Even after https://github.com/apache/datafusion/pull/20111, `simplify_const_expr` does unnecessary work, it can be pulled outside of the recursive transformation loop, and only calculated once for every expression subtree. The current implementation runs through the whole tree for every node that is evaluated in the main recursion in the loop, every time. -- 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]
