CTTY commented on code in PR #2232:
URL: https://github.com/apache/iceberg-rust/pull/2232#discussion_r2967160819


##########
crates/iceberg/src/catalog/mod.rs:
##########
@@ -98,6 +99,22 @@ pub trait Catalog: Debug + Sync + Send {
     /// Drop a table from the catalog, or returns error if it doesn't exist.
     async fn drop_table(&self, table: &TableIdent) -> Result<()>;
 
+    /// Drop a table from the catalog and delete the underlying table data.
+    ///
+    /// The default implementation loads the table metadata, drops the table
+    /// from the catalog, then deletes all associated data and metadata files
+    /// using [`drop_table_data`](utils::drop_table_data).
+    async fn purge_table(&self, table: &TableIdent) -> Result<()> {

Review Comment:
   This makes sense



-- 
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