[
https://issues.apache.org/jira/browse/HADOOP-13522?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15548035#comment-15548035
]
Akira Ajisaka commented on HADOOP-13522:
----------------------------------------
Thank you for updating the patch, [~algarbar].
1. Would you render the lines within 80 characters?
{code:title=FsPermission.java}
int octal = (n >>> 9 & 1)*1000 + (n >>> 6 & 7)*100 + (n >>> 3 & 7)*10 + (n
& 7);
{code}
{code:title=TestDFSShell.java}
assertTrue(out.toString(),
out.toString().contains(status1.getPermission().toString()));
{code}
2. Would you add an assertion for {{%a}} option?
{code}
assertTrue(out.toString(), out.toString().contains(
Short.toString(status1.getPermission().toShort())));
{code}
3. Would you add assertions for testFile2 as well?
(You don't need to set the assignee to me. In this project, assignee is a
developer who creates a patch.)
> Permissions on fs stat added
> ----------------------------
>
> Key: HADOOP-13522
> URL: https://issues.apache.org/jira/browse/HADOOP-13522
> Project: Hadoop Common
> Issue Type: Improvement
> Components: fs
> Affects Versions: 3.0.0-alpha1
> Reporter: Alex Garbarini
> Assignee: Akira Ajisaka
> Attachments: HADOOP-13522.001.patch, HADOOP-13522.002.patch,
> HADOOP-13522.003.patch, HADOOP-13522.004.patch, HADOOP-13522.005.patch,
> HADOOP-13522.006.patch
>
>
> This patch adds to fs/shell/Stat.java the missing options of %a and %A.
> FileStatus already contains the getPermission() method required for returning
> symbolic permissions. FsPermission contains the method to return the binary
> short, but nothing to present in standard Octal format.
> Most UNIX admins base their work on such standard octal permissions. Hence,
> this patch also introduces one tiny method to translate the toShort() return
> into octal.
> Build has already passed unit tests and javadoc.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
