[
https://issues.apache.org/jira/browse/HIVE-9474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14294085#comment-14294085
]
Szehon Ho commented on HIVE-9474:
---------------------------------
Hey Aihua, this looks good but a thought occurred to me. We would need to also
make sure the HDFS extendedAcl's are preserved in case of permission
inheritance (this is in additional to file permission). So the code would look
like:
{code}
HdfsFileStatus fullFileStatus = shim.getFullFileStatus(conf, fs,
location);
try {
shim.setFullFileStatus(conf, fullFileStatus, fs, location);
} catch (Exception e) {
LOG.warn("Error setting permissions of " + location, e);
}
{code}
You can check FileUtils as a reference. I believe we won't need to add an
additional test as by putting it in this base test, TestExtendedAcls will also
test this as well.
> truncate table changes permissions on the target
> ------------------------------------------------
>
> Key: HIVE-9474
> URL: https://issues.apache.org/jira/browse/HIVE-9474
> Project: Hive
> Issue Type: Bug
> Components: Query Processor
> Reporter: Aihua Xu
> Assignee: Aihua Xu
> Priority: Minor
> Fix For: 0.15.0
>
> Attachments: HIVE-9474.1.patch, HIVE-9474.2.patch
>
> Original Estimate: 4h
> Remaining Estimate: 4h
>
> Create a table test(a string);
> Hive> create table test(key string);
> Change the /user/hive/warehouse/test permission to something else other than
> the default, like 777.
> Hive> dfs -chmod 777 /user/hive/warehouse/test;
> Hive> dfs -ls -d /user/hive/warehouse/test;
> drwxrwxrwx - axu wheel 68 2015-01-26 18:45 /user/hive/warehouse/test
> Then truncate table test;
> Hive> truncate table test;
> The permission goes back to the default.
> hive> dfs -ls -d /user/hive/warehouse/test;
> drwxr-xr-x - axu wheel 68 2015-01-27 10:09 /user/hive/warehouse/test
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
