You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "John Zhuge (JIRA)" <ji...@apache.org> on 2016/06/23 21:13:16 UTC

[jira] [Created] (HADOOP-13315) FileContext#umask is not initialized properly

John Zhuge created HADOOP-13315:
-----------------------------------

             Summary: FileContext#umask is not initialized properly
                 Key: HADOOP-13315
                 URL: https://issues.apache.org/jira/browse/HADOOP-13315
             Project: Hadoop Common
          Issue Type: Bug
            Reporter: John Zhuge
            Assignee: John Zhuge
            Priority: Minor


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

{code:title=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);
  }
{code}

Proposal:
* Set {{umask}} to {{theUmask}}. Since the only caller {{getFileContext}} already passes the same value in {{theUmask}}, there is no change in behavior.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org