[jira] [Commented] (HDFS-6404) HttpFS should use a 000 umask for mkdir and create operations

2014-05-31 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-6404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14014690#comment-14014690
 ] 

Hudson commented on HDFS-6404:
--

FAILURE: Integrated in Hadoop-Mapreduce-trunk #1787 (See 
[https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1787/])
HDFS-6404. HttpFS should use a 000 umask for mkdir and create operations. 
(yoderme via tucu) (tucu: 
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1598668)
* 
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/main/java/org/apache/hadoop/lib/service/hadoop/FileSystemAccessService.java
* 
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/server/TestHttpFSServer.java
* /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt


> HttpFS should use a 000 umask for mkdir and create operations
> -
>
> Key: HDFS-6404
> URL: https://issues.apache.org/jira/browse/HDFS-6404
> Project: Hadoop HDFS
>  Issue Type: Bug
>Affects Versions: 2.4.0
>Reporter: Alejandro Abdelnur
>Assignee: Mike Yoder
> Fix For: 2.5.0
>
> Attachments: jira-HDFS-6404.patch
>
>
> The FileSystem created by HttpFS should use a 000 umask not to affect the 
> permissions set by the client as it is the responsibility of the client to 
> resolve the right permissions based on the client unmask.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HDFS-6404) HttpFS should use a 000 umask for mkdir and create operations

2014-05-31 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-6404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14014660#comment-14014660
 ] 

Hudson commented on HDFS-6404:
--

FAILURE: Integrated in Hadoop-Hdfs-trunk #1760 (See 
[https://builds.apache.org/job/Hadoop-Hdfs-trunk/1760/])
HDFS-6404. HttpFS should use a 000 umask for mkdir and create operations. 
(yoderme via tucu) (tucu: 
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1598668)
* 
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/main/java/org/apache/hadoop/lib/service/hadoop/FileSystemAccessService.java
* 
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/server/TestHttpFSServer.java
* /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt


> HttpFS should use a 000 umask for mkdir and create operations
> -
>
> Key: HDFS-6404
> URL: https://issues.apache.org/jira/browse/HDFS-6404
> Project: Hadoop HDFS
>  Issue Type: Bug
>Affects Versions: 2.4.0
>Reporter: Alejandro Abdelnur
>Assignee: Mike Yoder
> Fix For: 2.5.0
>
> Attachments: jira-HDFS-6404.patch
>
>
> The FileSystem created by HttpFS should use a 000 umask not to affect the 
> permissions set by the client as it is the responsibility of the client to 
> resolve the right permissions based on the client unmask.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HDFS-6404) HttpFS should use a 000 umask for mkdir and create operations

2014-05-31 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-6404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14014619#comment-14014619
 ] 

Hudson commented on HDFS-6404:
--

FAILURE: Integrated in Hadoop-Yarn-trunk #569 (See 
[https://builds.apache.org/job/Hadoop-Yarn-trunk/569/])
HDFS-6404. HttpFS should use a 000 umask for mkdir and create operations. 
(yoderme via tucu) (tucu: 
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1598668)
* 
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/main/java/org/apache/hadoop/lib/service/hadoop/FileSystemAccessService.java
* 
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/server/TestHttpFSServer.java
* /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt


> HttpFS should use a 000 umask for mkdir and create operations
> -
>
> Key: HDFS-6404
> URL: https://issues.apache.org/jira/browse/HDFS-6404
> Project: Hadoop HDFS
>  Issue Type: Bug
>Affects Versions: 2.4.0
>Reporter: Alejandro Abdelnur
>Assignee: Mike Yoder
> Fix For: 2.5.0
>
> Attachments: jira-HDFS-6404.patch
>
>
> The FileSystem created by HttpFS should use a 000 umask not to affect the 
> permissions set by the client as it is the responsibility of the client to 
> resolve the right permissions based on the client unmask.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HDFS-6404) HttpFS should use a 000 umask for mkdir and create operations

2014-05-30 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-6404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14013932#comment-14013932
 ] 

Hudson commented on HDFS-6404:
--

SUCCESS: Integrated in Hadoop-trunk-Commit #5633 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/5633/])
HDFS-6404. HttpFS should use a 000 umask for mkdir and create operations. 
(yoderme via tucu) (tucu: 
http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1598668)
* 
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/main/java/org/apache/hadoop/lib/service/hadoop/FileSystemAccessService.java
* 
/hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs-httpfs/src/test/java/org/apache/hadoop/fs/http/server/TestHttpFSServer.java
* /hadoop/common/trunk/hadoop-hdfs-project/hadoop-hdfs/CHANGES.txt


> HttpFS should use a 000 umask for mkdir and create operations
> -
>
> Key: HDFS-6404
> URL: https://issues.apache.org/jira/browse/HDFS-6404
> Project: Hadoop HDFS
>  Issue Type: Bug
>Affects Versions: 2.4.0
>Reporter: Alejandro Abdelnur
>Assignee: Mike Yoder
> Fix For: 2.5.0
>
> Attachments: jira-HDFS-6404.patch
>
>
> The FileSystem created by HttpFS should use a 000 umask not to affect the 
> permissions set by the client as it is the responsibility of the client to 
> resolve the right permissions based on the client unmask.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HDFS-6404) HttpFS should use a 000 umask for mkdir and create operations

2014-05-30 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-6404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14013746#comment-14013746
 ] 

Hadoop QA commented on HDFS-6404:
-

{color:green}+1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12647602/jira-HDFS-6404.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 1 new 
or modified test files.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  There were no new javadoc warning messages.

{color:green}+1 eclipse:eclipse{color}.  The patch built with 
eclipse:eclipse.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 1.3.9) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 core tests{color}.  The patch passed unit tests in 
hadoop-hdfs-project/hadoop-hdfs-httpfs.

{color:green}+1 contrib tests{color}.  The patch passed contrib unit tests.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/7013//testReport/
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/7013//console

This message is automatically generated.

> HttpFS should use a 000 umask for mkdir and create operations
> -
>
> Key: HDFS-6404
> URL: https://issues.apache.org/jira/browse/HDFS-6404
> Project: Hadoop HDFS
>  Issue Type: Bug
>Affects Versions: 2.4.0
>Reporter: Alejandro Abdelnur
>Assignee: Mike Yoder
> Attachments: jira-HDFS-6404.patch
>
>
> The FileSystem created by HttpFS should use a 000 umask not to affect the 
> permissions set by the client as it is the responsibility of the client to 
> resolve the right permissions based on the client unmask.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HDFS-6404) HttpFS should use a 000 umask for mkdir and create operations

2014-05-29 Thread Alejandro Abdelnur (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-6404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14013337#comment-14013337
 ] 

Alejandro Abdelnur commented on HDFS-6404:
--

Patch looks good, a couple of nits:

* FileSystemAccessService.java: comment should read: // Force-clear server-side 
umask to make HttpFS match WebHDFS behavior
* TestHttpFSServer.java: don't use * imports.

+1 after they are fixed.

> HttpFS should use a 000 umask for mkdir and create operations
> -
>
> Key: HDFS-6404
> URL: https://issues.apache.org/jira/browse/HDFS-6404
> Project: Hadoop HDFS
>  Issue Type: Bug
>Affects Versions: 2.4.0
>Reporter: Alejandro Abdelnur
>Assignee: Mike Yoder
> Attachments: jira-HDFS-6404.patch
>
>
> The FileSystem created by HttpFS should use a 000 umask not to affect the 
> permissions set by the client as it is the responsibility of the client to 
> resolve the right permissions based on the client unmask.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HDFS-6404) HttpFS should use a 000 umask for mkdir and create operations

2014-05-29 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-6404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14013186#comment-14013186
 ] 

Hadoop QA commented on HDFS-6404:
-

{color:green}+1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12647497/jira-HDFS-6404.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:green}+1 tests included{color}.  The patch appears to include 1 new 
or modified test files.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 javadoc{color}.  There were no new javadoc warning messages.

{color:green}+1 eclipse:eclipse{color}.  The patch built with 
eclipse:eclipse.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 1.3.9) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 core tests{color}.  The patch passed unit tests in 
hadoop-hdfs-project/hadoop-hdfs-httpfs.

{color:green}+1 contrib tests{color}.  The patch passed contrib unit tests.

Test results: 
https://builds.apache.org/job/PreCommit-HDFS-Build/7011//testReport/
Console output: https://builds.apache.org/job/PreCommit-HDFS-Build/7011//console

This message is automatically generated.

> HttpFS should use a 000 umask for mkdir and create operations
> -
>
> Key: HDFS-6404
> URL: https://issues.apache.org/jira/browse/HDFS-6404
> Project: Hadoop HDFS
>  Issue Type: Bug
>Affects Versions: 2.4.0
>Reporter: Alejandro Abdelnur
>Assignee: Mike Yoder
> Attachments: jira-HDFS-6404.patch
>
>
> The FileSystem created by HttpFS should use a 000 umask not to affect the 
> permissions set by the client as it is the responsibility of the client to 
> resolve the right permissions based on the client unmask.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HDFS-6404) HttpFS should use a 000 umask for mkdir and create operations

2014-05-16 Thread bc Wong (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-6404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=1423#comment-1423
 ] 

bc Wong commented on HDFS-6404:
---

If the REST call comes with permission, then surely use the one specified by 
the client. But if the call doesn't specify permission, then it'd seem more 
correct to use fs.permissions.umask-mode.

Basically, I don't think that the client is responsible for resolving the right 
permissions based on the client umask, because the "permission" param is 
optional in the REST API.

> HttpFS should use a 000 umask for mkdir and create operations
> -
>
> Key: HDFS-6404
> URL: https://issues.apache.org/jira/browse/HDFS-6404
> Project: Hadoop HDFS
>  Issue Type: Bug
>Affects Versions: 2.4.0
>Reporter: Alejandro Abdelnur
>Assignee: Alejandro Abdelnur
>
> The FileSystem created by HttpFS should use a 000 umask not to affect the 
> permissions set by the client as it is the responsibility of the client to 
> resolve the right permissions based on the client unmask.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HDFS-6404) HttpFS should use a 000 umask for mkdir and create operations

2014-05-14 Thread Alejandro Abdelnur (JIRA)

[ 
https://issues.apache.org/jira/browse/HDFS-6404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13998253#comment-13998253
 ] 

Alejandro Abdelnur commented on HDFS-6404:
--

The fix should be done in the {{FileSystemAccessService.java}} class in the 
{{getFileSystemConfiguration()}} method, the configuration should be seeded 
with {{conf.set(FsPermission.UMASK_LABEL, "000")}}


> HttpFS should use a 000 umask for mkdir and create operations
> -
>
> Key: HDFS-6404
> URL: https://issues.apache.org/jira/browse/HDFS-6404
> Project: Hadoop HDFS
>  Issue Type: Bug
>Affects Versions: 2.4.0
>Reporter: Alejandro Abdelnur
>Assignee: Alejandro Abdelnur
>
> The FileSystem created by HttpFS should use a 000 umask not to affect the 
> permissions set by the client as it is the responsibility of the client to 
> resolve the right permissions based on the client unmask.



--
This message was sent by Atlassian JIRA
(v6.2#6252)