[HADOOP-13612] FileSystem static mkdirs(FS, path, permissions) to invoke FS.mkdirs(path, permissions)

• Type: Improvement
• Status: Patch Available
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 2.7.3
• Component/s: fs
• Assignee: Steve Loughran
• Reporter: Steve Loughran
• Created: 14/Sep/16 13:58
• Updated: 25/Jun/18 19:40

Description

Currently FileSystem's static mkdirs(FileSystem fs, Path dir, FsPermission permission) creates the directory in a two step operation

    // create the directory using the default permission
    boolean result = fs.mkdirs(dir);
    // set its permission to be the supplied one
    fs.setPermission(dir, permission);

this isn't atomic and creates a risk of race/security conditions. This code is used in production

Better to simply forward to mkdirs(path, permissions).

is there any reason to not do that?

Issue Links

• relates to: HADOOP-1873 User permissions for Map/Reduce