[jira] [Updated] (HIVE-22599) Query results cache: 733 permissions check is not necessary
[ https://issues.apache.org/jira/browse/HIVE-22599?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Dere updated HIVE-22599: -- Fix Version/s: 4.0.0 Resolution: Fixed Status: Resolved (was: Patch Available) Committed to master > 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 > Fix For: 4.0.0 > > 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)
[jira] [Updated] (HIVE-22599) Query results cache: 733 permissions check is not necessary
[ https://issues.apache.org/jira/browse/HIVE-22599?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Dere updated HIVE-22599: -- Attachment: HIVE-22599.1.patch > 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)
[jira] [Updated] (HIVE-22599) Query results cache: 733 permissions check is not necessary
[ https://issues.apache.org/jira/browse/HIVE-22599?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Dere updated HIVE-22599: -- Attachment: (was: HIVE-22599.1.patch) > 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)
[jira] [Updated] (HIVE-22599) Query results cache: 733 permissions check is not necessary
[ https://issues.apache.org/jira/browse/HIVE-22599?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jason Dere updated HIVE-22599: -- Attachment: HIVE-22599.1.patch Status: Patch Available (was: Open) > 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)