tshauck commented on PR #9729: URL: https://github.com/apache/arrow-datafusion/pull/9729#issuecomment-2018565479
Thanks for the feedback, I reverted the extension changes, and now it only has the example plus making `Parser` public on `DFParser`. To your point later, there's quite a bit within `sql_statement_to_plan` (and functions it calls) that would be nice to not have to replicate, but perhaps going through the work of using this change in my code will make it clearer what those changes should be. Things like resolving name, handling subqueries, etc. -- 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]
