ajantha-bhat commented on PR #6786:
URL: https://github.com/apache/iceberg/pull/6786#issuecomment-1424471540

   > This is what i'm saying, if we want an api which does not treat it as an 
Iceberg table first I think we need that as a Catalog API. If Drop Table needs 
the table to be an iceberg table to drop then I think we either need a new api, 
or a flag like "force" which attempts to drop the table even if there is an 
exception.
   
   When the `purge=false` for drop table API, it doesn't have to be an Iceberg 
table. It works just based on table identifier name. 
   For example, 
https://github.com/apache/iceberg/blob/master/hive-metastore/src/main/java/org/apache/iceberg/hive/HiveCatalog.java#L189-L198
   
   Also, Trino also doing the same and it is using the same API that I am using 
here. 
   https://github.com/trinodb/trino/pull/15874/files


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