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
{{ message }}
Repository navigation
Create Struct table with explicit type and name #10207
I think upstream sqlparser lib already support similar syntax
create table t (s struct<n int, s varchar>);
But only bigquery dialect support this syntax.
However in this PR where struct syntax is is added they also decided to support this syntax for both Bigquery and GenericDialect, and i think they forgot to add | GenericDialect for the column definition parsing logic here
Activity
duongcongtoai commentedon Apr 27, 2024
i would like to work on this
duongcongtoai commentedon Apr 28, 2024
1st item:
I think upstream sqlparser lib already support similar syntax
But only bigquery dialect support this syntax.
However in this PR where struct syntax is is added they also decided to support this syntax for both Bigquery and GenericDialect, and i think they forgot to add | GenericDialect for the column definition parsing logic here
Ticket to follow: apache/datafusion-sqlparser-rs#1240