nathanb9 commented on issue #22676: URL: https://github.com/apache/datafusion/issues/22676#issuecomment-4596943986
> materialization is useful not only for CTEs but for duplicated subplans in general Makes sense, generalize this more. I was going to say "why use subqueries over CTEs when reusing? Besides if they're calling volatile function?" but I guess we should optimize for SQL expressiveness regardless bc maybe thats just how I usually use SQL 😆 > what about not touching the SQL planner at all but add a logical Besides to support the `MATERIALIZED`/`AS NOT MATERALIZED` I think the change there will be minimal -- 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]
