You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm not sure if this should parse, Postgres for example also fails, but if possible it would be nice if this at least parsed if not executed correctly.
I tested this in datafusion-cli, but originally found it using Rust:
> SELECT 1 in (); 🤔 Invalid statement: sql parser error: Expected: an expression:, found: )
The text was updated successfully, but these errors were encountered:
@alamb@iffyio The empty () should be disallowed in many dialects, but I'm not sure if it's good to add an option to allow this in the SQL parser. This seems reasonable from my side since this behavior can be determined by the downstream, and what we should guarantee is that it won't break the old behavior.
I'm not sure if this should parse, Postgres for example also fails, but if possible it would be nice if this at least parsed if not executed correctly.
I tested this in datafusion-cli, but originally found it using Rust:
The text was updated successfully, but these errors were encountered: