zxl-333 commented on code in PR #5975:
URL: https://github.com/apache/hive/pull/5975#discussion_r2215074657
##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java:
##########
@@ -3047,7 +3047,8 @@ private boolean drop_table_core(final RawStore ms, final
String catName, final S
tableDataShouldBeDeleted = checkTableDataShouldBeDeleted(tbl,
deleteData);
if (tableDataShouldBeDeleted && tbl.getSd().getLocation() != null) {
tblPath = new Path(tbl.getSd().getLocation());
- if (!wh.isWritable(tblPath.getParent())) {
+ // HIVE-28804 drop table user should have table path and parent path
permission
Review Comment:
1. The directory permissions for the /warehouse/tablespace/managed/hive/ are
set to **777**
2. The **test_20250718** database was created using **user_1**, and the
directory permissions and HDFS path permissions for the library are both set to
**777**.
/warehouse/tablespace/managed/hive/test_20250718.db
3. Use the **user_1** user to create a test table, and set the HDFS
directory permission of the table to **755**. create table test_1(k string);
4. Switch to the **user_2** account to delete the table **test_1**. At this
point, you will notice that the metastore database of the **test_1** table has
been deleted, but the HDFS directory has not been deleted. By checking the
metastore log, you will find the abnormal stack information I provided.
Under normal circumstances, user_2 should not be able to delete test_1.
--
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: gitbox-unsubscr...@hive.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
