[HIVE-22599] Query results cache: 733 permissions check is not necessary

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.0.0-alpha-1
• Assignee: Jason Dere
• Reporter: Jason Dere
• Created: 09/Dec/19 03:11
• Updated: 17/Nov/22 08:51
• Resolved: 13/Dec/19 23:33

Description

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.