iffyio commented on code in PR #1890:
URL: 
https://github.com/apache/datafusion-sqlparser-rs/pull/1890#discussion_r2151983259


##########
tests/sqlparser_bigquery.rs:
##########
@@ -2407,3 +2459,16 @@ fn test_any_type() {
 fn test_any_type_dont_break_custom_type() {
     bigquery_and_generic().verified_stmt("CREATE TABLE foo (x ANY)");
 }
+
+#[test]
+fn test_allow_options_in_struct_field() {

Review Comment:
   ```suggestion
   fn test_struct_field_options() {
   ```



##########
src/ast/mod.rs:
##########
@@ -428,14 +428,22 @@ impl fmt::Display for Interval {
 pub struct StructField {
     pub field_name: Option<Ident>,
     pub field_type: DataType,
+    // BigQuery supports options for struct fields.
+    // see [BigQuery]: 
https://cloud.google.com/bigquery/docs/reference/standard-sql/data-definition-language#column_name_and_column_schema

Review Comment:
   ```suggestion
       /// Struct field options.
       /// see [BigQuery]: 
https://cloud.google.com/bigquery/docs/reference/standard-sql/data-definition-language#column_name_and_column_schema
   ```



-- 
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

Reply via email to