PokIsemaine commented on issue #14514: URL: https://github.com/apache/datafusion/issues/14514#issuecomment-2644742294
> Hello [@PokIsemaine](https://github.com/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 https://github.com/apache/datafusion/pull/7622, if the syntax was originally not supported by [sqlparser](https://github.com/apache/datafusion-sqlparser-rs), do I need to first submit a PR in the [sqlparser](https://github.com/apache/datafusion-sqlparser-rs) repository and then proceed with the implementation in datafusion? Or should we just go ahead with the implementation in datafusion? -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
