[HADOOP-13315] FileContext#umask is not initialized properly

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.6.0
• Fix Version/s: 2.8.0, 3.0.0-alpha1
• Target Version/s: 2.8.0
• Assignee: John Zhuge
• Reporter: John Zhuge
• Created: 23/Jun/16 21:12
• Updated: 06/Jan/17 08:42
• Resolved: 12/Jul/16 23:54

Description

Notice field umask is not set to parameter theUmask and theUmask is unused.

FileContext.java

  private FileContext(final AbstractFileSystem defFs,
    final FsPermission theUmask, final Configuration aConf) {
    defaultFS = defFs;
    umask = FsPermission.getUMask(aConf);
    conf = aConf;
...
  public static FileContext getFileContext(final AbstractFileSystem defFS,
                    final Configuration aConf) {
    return new FileContext(defFS, FsPermission.getUMask(aConf), aConf);
  }

Proposal:

• Set umask to theUmask. Since the only caller getFileContext already passes the same value in theUmask, there is no change in behavior.