[GitHub] [iceberg] pvary edited a comment on issue #2554: Cannot Drop Table Created with HiveIcebergStorageHandler Enabled but Metadata.json is Missing

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

pvary edited a comment on issue #2554:
URL: https://github.com/apache/iceberg/issues/2554#issuecomment-836762324


   @marton-bod: I would prefer your first suggestion:
   - Log a warn message
   - `DROP TABLE` should be successful
   
   There are automatic clients who might expect the table to remain if the `DROP TABLE` command is not successful. This could be confusing. We either do not drop a table metadata and return an error, or drop the table metadata and return successfully. At least that was the reasoning behind our decision in the aforementioned case.
   
   > The next issue we kept hitting is that there were no iceberg jars on the hive class-path which is a problem because you can always set the storage handler, but once set, you can't remove the property without the jars on the class-path
   
   This is another story might worth another issue, if we are not able to drop the table even from Spark, or Java clients


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

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