[GitHub] [iceberg] pvary commented on pull request #1240: Default warehouse location of a table should be a subdirectory in database location

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

pvary commented on pull request #1240:
URL: https://github.com/apache/iceberg/pull/1240#issuecomment-740008622


   > If it is set to false here, there will be a situation where the hdfs file directory will not be deleted when the flink sql client is deleting the database. What is the reason for this setting to false?
   
   @XuQianJin-Stars: This PR has not changed the behavior of dropping namespaces. The highlighted code was added with #919
   
   Independently of this IMO the best place for these questions would be the dev list, but I guess that we wanted to be conservative about dropping a namespace to prevent accidentally removing wrong directories and with them important data.
   
   


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