alamb opened a new issue, #16242: URL: https://github.com/apache/datafusion/issues/16242
### Is your feature request related to a problem or challenge? While I was reviewing https://github.com/apache/datafusion/pull/16217 I did some profiling on the planning benchmarks It looks like more than 2% of the planning time is spent trying to plan `UNEST` in the planner, but none of the benchmark queries have any `UNEST` in them <img width="1366" alt="Image" src="https://github.com/user-attachments/assets/a70873bf-eb26-43a1-8881-b88f81fc7889"; /> ### Describe the solution you'd like Since tpch q10 doesn't have any `unnest` statements in it, spending 2% of the planning time trying to plan them makes no sense. I think we can improve the planning performance significantly ### Describe alternatives you've considered To reproduce: ```shell cargo bench --profile=profiling --bench sql_planner physical_plan_tpch_q10 ``` Then [use `samply` as described in the docs](https://datafusion.apache.org/library-user-guide/profiling.html#profiling-using-samply-cross-platform-profiler): ```shell samply record target/profiling/deps/sql_planner-88f105135556ad27 --bench physical_plan_tpch_q10 ``` (note you will have to update the actual binary name based on the output of running `cargo bench` ### Additional context This would be a good issue for someone who was reasonably familiar with Rust and DataFusion and wanted to work on optimization -- 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...@datafusion.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
