alamb commented on issue #5492: URL: https://github.com/apache/datafusion/issues/5492#issuecomment-2899282132
> Thanks [@duongcongtoai](https://github.com/duongcongtoai) ! This is a very good idea! I also think we can start with simple unnest, we may need to introduce some DuckDB structures: new logical plan/expr(`DelimScan`, ...), some new structures (`delim_offset`, `has_correlated_expressions`, ..) By trying to refactor simple unnest, we can discover some limitations of DataFusion and adjust our follow-up plans in a timely manner. I don't fully follow what this is proposing Are you proposing to add new `LogicalPlan::DelimScan` variant? It was not entirely clear to me reading this thread if the DelimScan is something that anything other than the decorrelation optimizer pass would use -- 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 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
