Repository navigation
[EPIC] Support converting Exprs and LogicalPlans --> SQL Strings #9494
Description
This ticket tracks various tickets related to converting Exprs and LogicalPlans to/from SQL strings
- Support parsing SQL strings to
Exprs #8736 - [EPIC] Support LogicalPlan -->
SQL String#8661 - Complete support for
Expr --> String#9726 - Basic support for converting
Exprto SQL string #9495 - Add an example of how to use the SQL parser/unparser API #10524
- Add an example of how to convert LogicalPlan to/from SQL Strings #10550
- Make SQL strings generated from
Exprs "prettier" #10557 - Make SQL strings generated from Exprs even "prettier" #10633
Activity
devinjdangelo commentedon Mar 19, 2024
I think that we now have a solid enough foundation on these tasks that it would be a good first issue for a new contributor (#9495 especially). An individual or small number of logical
Exprs could be handled in a stand alone PR with a test or two. I think this project is a great way to get more familiar with DataFusion's core data structures and optimization algorithms without already having expertise in database internals.alamb commentedon Mar 19, 2024
@devinjdangelo do you think we should file explicit follow on tickets, or do you think the existing PRs / documentation is clear enough about the ask (for someone who has no experience with DataFusion)?
I can file tickets if you prefer
devinjdangelo commentedon Mar 19, 2024
@alamb I don't think we should file a ticket for every remaining
Exprvariant, but we could edit the description of #9495 to list the remaining unimplemented variants and make clear that we are looking for help for anyone interested to implement even just 1 at a time in a PR + write a test.Here is the remaining list:
alamb commentedon Mar 21, 2024
Thank yoU @devinjdangelo -- I filed #9726
[-][EPIC] Collection of SQL to/from LogicalPlan/Expr tickets[/-][+][EPIC] Support converting Exprs and LogicalPlans --> SQL Strings[/+]alamb commentedon Nov 22, 2024
Let's claim this is complete 🎉 All the existing subtasks are closed and now we are working on this in our ongoing operations