yutoacts commented on a change in pull request #35239:
URL: https://github.com/apache/spark/pull/35239#discussion_r791538241
##########
File path: docs/sql-ref-syntax-ddl-alter-table.md
##########
@@ -309,6 +397,116 @@ DESC StudentInfo;
| age| int| NULL|
+-----------------------+---------+-------+
+-- Drop columns of a table
+DESC StudentInfo;
++-----------------------+---------+-------+
+| col_name|data_type|comment|
++-----------------------+---------+-------+
+| name| string| NULL|
+| rollno| int| NULL|
+| LastName| string| NULL|
+| DOB|timestamp| NULL|
+| age| int| NULL|
+|# Partition Information| | |
+| # col_name|data_type|comment|
+| age| int| NULL|
++-----------------------+---------+-------+
+
+ALTER TABLE StudentInfo DROP columns (LastName, DOB);
Review comment:
@cloud-fan Thanks for the review. right, those statements I added
except `RECOVER PARTITIONS` are not supported by builtin sources but users can
technically use them by their custom file sources so I think they should be
documented. I mentioned in a doc that they are only supported with v2 tables.
Do you think they should not be in example?
--
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]
