[GitHub] [iceberg] samredai commented on a diff in pull request #4746: Docs: Add syntax description for deleting empty tables

[GitBox <gi...@apache.org>]

samredai commented on code in PR #4746:
URL: https://github.com/apache/iceberg/pull/4746#discussion_r883738477


##########
docs/spark/spark-ddl.md:
##########
@@ -134,6 +134,11 @@ To delete a table, run:
 DROP TABLE prod.db.sample
 ```
 
+To delete an empty table use `PURGE`, run:
+
+```sql
+DROP TABLE prod.db.sample PURGE
+```

Review Comment:
   I like that wording. I think the additional statement could go in a hint box though. What do ya'll think of this?
   
   > To delete a table and its underlying data, use the `PURGE keyword.
   > ```sql
   > DROP TABLE prod.db.sample PURGE
   > ```
   > {{< hint info >}}
   > The `PURGE` flag can also be used to remove an empty table from its corresponding catalog if the table's metadata files have been removed by some other process and cannot be recovered.
   > {{< /hint >}}



-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org