saihemanth-cloudera commented on code in PR #5997:
URL: https://github.com/apache/hive/pull/5997#discussion_r2286093248


##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java:
##########
@@ -3050,18 +3050,6 @@ private boolean drop_table_core(final RawStore ms, final 
String catName, final S
       firePreEvent(new PreDropTableEvent(tbl, deleteData, this));
 
       tableDataShouldBeDeleted = checkTableDataShouldBeDeleted(tbl, 
deleteData);
-      if (tableDataShouldBeDeleted && tbl.getSd().getLocation() != null) {

Review Comment:
   @zxl-333 -- Can you please give me a detailed repro of this issue (probably 
a doc with screenshots with ranger policies, core-site.xml (in Hadoop conf) 
etc.), as previously asked by @deniskuzZ as well.
   Because I cannot get this whatsoever
   
   > then after using ranger to control HDFS permissions, it is still necessary 
to grant HDFS ACL permissions to the user.
   
   If you are using to control HDFS permissions, then you don't need to grant 
the end user the required permissions. 
   I'm guessing that you are missing this config in your core-site.xml of 
Hadoop service:
   `  <property>
           <name>hadoop.proxyuser.hive.hosts</name>
           <value>*</value>
       </property>`
    `   <property>
           <name>hadoop.proxyuser.hive.groups</name>
           <value>*</value>
       </property>`



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

Reply via email to