kiszk commented on a change in pull request #28451:
URL: https://github.com/apache/spark/pull/28451#discussion_r420203845
##########
File path: docs/sql-ref-syntax-ddl-create-table-hiveformat.md
##########
@@ -36,67 +36,54 @@ CREATE [ EXTERNAL ] TABLE [ IF NOT EXISTS ] table_identifier
[ LOCATION path ]
[ TBLPROPERTIES ( key1=val1, key2=val2, ... ) ]
[ AS select_statement ]
-{% endhighlight %}
+```
Note that, the clauses between the columns definition clause and the AS SELECT
clause can come in
as any order. For example, you can write COMMENT table_comment after
TBLPROPERTIES.
### Parameters
-<dl>
- <dt><code><em>table_identifier</em></code></dt>
- <dd>
- Specifies a table name, which may be optionally qualified with a database
name.<br><br>
- <b>Syntax:</b>
- <code>
- [ database_name. ] table_name
- </code>
- </dd>
-</dl>
-
-<dl>
- <dt><code><em>EXTERNAL</em></code></dt>
- <dd>Table is defined using the path provided as LOCATION, does not use
default location for this table.</dd>
-</dl>
-
-<dl>
- <dt><code><em>PARTITIONED BY</em></code></dt>
- <dd>Partitions are created on the table, based on the columns specified.</dd>
-</dl>
-
-<dl>
- <dt><code><em>ROW FORMAT</em></code></dt>
- <dd>SERDE is used to specify a custom SerDe or the DELIMITED clause in order
to use the native SerDe.</dd>
-</dl>
-
-<dl>
- <dt><code><em>STORED AS</em></code></dt>
- <dd>File format for table storage, could be TEXTFILE, ORC,
PARQUET,etc.</dd>
-</dl>
-
-<dl>
- <dt><code><em>LOCATION</em></code></dt>
- <dd>Path to the directory where table data is stored, Path to the directory
where table data is stored, which could be a path on distributed storage like
HDFS, etc.</dd>
-</dl>
-
-<dl>
- <dt><code><em>COMMENT</em></code></dt>
- <dd>A string literal to describe the table.</dd>
-</dl>
-
-<dl>
- <dt><code><em>TBLPROPERTIES</em></code></dt>
- <dd>A list of key-value pairs that is used to tag the table definition.</dd>
-</dl>
-
-<dl>
- <dt><code><em>AS select_statement</em></code></dt>
- <dd>The table is populated using the data from the select statement.</dd>
-</dl>
+* **table_identifier**
+
+ Specifies a table name, which may be optionally qualified with a database
name.
+
+ **Syntax:** `[ database_name. ] table_name`
+
+* **EXTERNAL**
+
+ Table is defined using the path provided as LOCATION, does not use default
location for this table.
+
+* **PARTITIONED BY**
+
+ Partitions are created on the table, based on the columns specified.
+
+* **ROW FORMAT**
+
+ SERDE is used to specify a custom SerDe or the DELIMITED clause in order
to use the native SerDe.
+
+* **STORED AS**
+
+ File format for table storage, could be TEXTFILE, ORC, PARQUET,etc.
Review comment:
nit: `, etc`
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
