Steve Loughran commented on HADOOP-12774:

Patch 003: Address Chris's comments about using parent class fields. Also, 
given that the first contructor for S3AFileStatus is only for directories, the 
isdir parameter is always true, hence superflous. Cut and add javadocs for both 

Tested: s3a ireland, all well except for some intermittent consistency failures 
in {{ITestS3AContractRootDir.testRmEmptyRootDirNonRecursive}}. need to file a 
JIRA there

> s3a should use UGI.getCurrentUser.getShortname() for username
> -------------------------------------------------------------
>                 Key: HADOOP-12774
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12774
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>    Affects Versions: 2.7.2
>            Reporter: Steve Loughran
>            Assignee: Steve Loughran
> S3a users {{System.getProperty("user.name")}} to get the username for the 
> homedir. This is wrong, as it doesn't work on a YARN app where the identity 
> is set by HADOOP_USER_NAME, or in a doAs clause.
> Obviously, {{UGI.getCurrentUser.getShortname()}} provides that name, 
> everywhere. 
> This is a simple change in the source, though testing is harder ... probably 
> best to try in a doAs

This message was sent by Atlassian JIRA

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

Reply via email to