-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[EPIC] DuckDB-Inspired Feature Enhancements #14514
Comments
Hello @berkaysynnada , I am interested in both I haven't deeply explored the core code of If there are any suitable |
Hello @PokIsemaine. I have an idea that hasn't been issued yet: implementing "ORDER BY ALL." Although this feature is not part of standard sql, duckdb supports it https://duckdb.org/docs/sql/query_syntax/orderby.html#order-by-all If you're looking for a warm-up project, implementing it in datafusion would be a great start. I suggest familiarizing yourself with the various parts of the codebase rather than diving straight into the core features. For guidance, you might want to look at how "group by all" is handled. |
Thanks for your advice! I noticed #7622, if the syntax was originally not supported by sqlparser, do I need to first submit a PR in the sqlparser repository and then proceed with the implementation in datafusion? Or should we just go ahead with the implementation in datafusion? |
Yes, you probably need to update |
apache/datafusion-sqlparser-rs#1724 |
@PokIsemaine Awesome! I have not even noticed yet you open the sqlparser PR 😅 I hope it won't take much time to release the new version. |
While I'm investigating some possible project ideas for GSoC 2025, I took a look at some cool DuckDB features and want to create issues for similar enhancements in datafusion. These features will improve datafusion's usability and performance.
(Note: Lambda support is tracked in #14205.)
There are also smaller-scale, but still very cool, features in this list:
https://duckdb.org/docs/sql/dialect/friendly_sql.html
Anyone is welcome to undertake any missing one from that list.
The text was updated successfully, but these errors were encountered: