[
https://issues.apache.org/jira/browse/HDFS-10650?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15384940#comment-15384940
]
John Zhuge commented on HDFS-10650:
-----------------------------------
Thanks [~xiaochen].
This jira came out of discussion
https://issues.apache.org/jira/browse/HDFS-6962?focusedCommentId=15383249&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15383249.
[~cnauroth], do you know the current behavior is a bug or intentional?
bq. why remove the null check?
My bug. I will fix it.
> DFSClient#mkdirs and DFSClient#primitiveMkdir should use default directory
> permission
> -------------------------------------------------------------------------------------
>
> Key: HDFS-10650
> URL: https://issues.apache.org/jira/browse/HDFS-10650
> Project: Hadoop HDFS
> Issue Type: Bug
> Affects Versions: 2.6.0
> Reporter: John Zhuge
> Assignee: John Zhuge
> Priority: Minor
> Attachments: HDFS-10650.001.patch
>
>
> These 2 DFSClient methods should use default directory permission to create a
> directory.
> {code:java}
> public boolean mkdirs(String src, FsPermission permission,
> boolean createParent) throws IOException {
> if (permission == null) {
> permission = FsPermission.getDefault();
> }
> {code}
> {code:java}
> public boolean primitiveMkdir(String src, FsPermission absPermission,
> boolean createParent)
> throws IOException {
> checkOpen();
> if (absPermission == null) {
> absPermission =
> FsPermission.getDefault().applyUMask(dfsClientConf.uMask);
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
