Re: [PR] Implement FROM-first selects [datafusion-sqlparser-rs]

2025-02-08 Thread via GitHub
iffyio merged PR #1713: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1713 -- 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

Re: [PR] Implement FROM-first selects [datafusion-sqlparser-rs]

2025-02-08 Thread via GitHub
mitsuhiko commented on code in PR #1713: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1713#discussion_r1947940772 ## tests/sqlparser_duckdb.rs: ## @@ -246,6 +246,67 @@ fn test_create_table_macro() { assert_eq!(expected, macro_); } +#[test] +fn test_select

Re: [PR] Implement FROM-first selects [datafusion-sqlparser-rs]

2025-02-07 Thread via GitHub
iffyio commented on code in PR #1713: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1713#discussion_r1947123514 ## tests/sqlparser_duckdb.rs: ## @@ -246,6 +246,67 @@ fn test_create_table_macro() { assert_eq!(expected, macro_); } +#[test] +fn test_select_fr

[PR] Implement FROM-first selects [datafusion-sqlparser-rs]

2025-02-06 Thread via GitHub
mitsuhiko opened a new pull request, #1713: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1713 This implements "FROM-first" selects. DuckDB and Clickhouse support this syntax where `FROM` can be placed before `SELECT`. To disambiguate these cases for formatting I added a new