alamb commented on PR #9606: URL: https://github.com/apache/arrow-datafusion/pull/9606#issuecomment-1999396476
In general, I don't think it will be possible to convert all `LogicalPlan`s to SQL as LogicalPlans can be built programatically and some will perfom calculations that are not possible to express in SQL Therefore, I think the best testing strategy is to do round trip tests that verify any pattern the DataFusion SQL planner makes can be converted back to SQL > This case will of course fail in the PR. I think that is OK, as we can work on support for more complex aggregations and additional edge cases we find in future PRs. I think this is a wise strategy. -- 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]
