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: [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]
