Github user viirya commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20133#discussion_r159150554
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
    @@ -384,22 +384,31 @@ class SparkSqlAstBuilder(conf: SQLConf) extends 
AstBuilder(conf) {
        *   CREATE [TEMPORARY] TABLE [IF NOT EXISTS] [db_name.]table_name
        *   USING table_provider
        *   [OPTIONS table_property_list]
    -   *   [PARTITIONED BY (col_name, col_name, ...)]
    -   *   [CLUSTERED BY (col_name, col_name, ...)
    -   *    [SORTED BY (col_name [ASC|DESC], ...)]
    -   *    INTO num_buckets BUCKETS
    -   *   ]
    -   *   [LOCATION path]
    -   *   [COMMENT table_comment]
    -   *   [TBLPROPERTIES (property_name=property_value, ...)]
    +   *   create_table_clauses
        *   [[AS] select_statement];
    +   *
    +   *   create_table_clauses (order insensitive):
    +   *     [PARTITIONED BY (col_name, col_name, ...)]
    --- End diff --
    
    Isn't `[OPTIONS table_property_list]` one of `create_table_clauses`?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to