[
https://issues.apache.org/jira/browse/HIVE-22599?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16995423#comment-16995423
]
Gopal Vijayaraghavan commented on HIVE-22599:
---------------------------------------------
LGTM - +1
> Query results cache: 733 permissions check is not necessary
> -----------------------------------------------------------
>
> Key: HIVE-22599
> URL: https://issues.apache.org/jira/browse/HIVE-22599
> Project: Hive
> Issue Type: Bug
> Reporter: Jason Dere
> Assignee: Jason Dere
> Priority: Major
> Attachments: HIVE-22599.1.patch
>
>
> The query results cache initialization makes a call to
> Utilties.ensurePathIsWritable(), which checks the results cache directory for
> 733 permissions (default cache dir is
> {{/tmp/hive/_resultscache_).}}
> The 733 permissions (at least the 033 part) are not actually necessary - we
> actually don't really want the results cache directory to be world-writable,
> and the subdirectories we create within this one are actually done with 700
> perms. So I think the call to Utilties.ensurePathIsWritable() can be removed.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
