huaxingao commented on a change in pull request #35239:
URL: https://github.com/apache/spark/pull/35239#discussion_r793252862



##########
File path: docs/sql-ref-syntax-ddl-alter-table.md
##########
@@ -75,6 +75,50 @@ ALTER TABLE table_identifier ADD COLUMNS ( col_spec [ , ... 
] )
 
     Specifies the columns to be added.
 
+### DROP COLUMNS
+
+`ALTER TABLE DROP COLUMNS` statement drops mentioned columns from an existing 
table.
+Note that this statement is only supported with v2 tables.
+
+
+#### Syntax
+
+```sql
+ALTER TABLE table_identifier DROP COLUMNS ( col_name1, ... )
+```

Review comment:
       I just realized that previously we used `col_spec` for add column. I 
think `col_name` is more accurate, because `col_spec` makes me think it is 
column name + data type. Could you please check if other doc use `col_spec` 
too? If yes, we probably want to use `col_spec` to be consistent.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to