[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16113247#comment-16113247 ] Manoj Govindassamy commented on HADOOP-14223: - [~vishwajeet.dusane], Thanks for reporting the test issues and for the debugging. Much appreciated. > Extend FileStatus#toString() to include details like Erasure Coding and > Encryption > -- > > Key: HADOOP-14223 > URL: https://issues.apache.org/jira/browse/HADOOP-14223 > Project: Hadoop Common > Issue Type: Bug > Components: fs >Affects Versions: 3.0.0-alpha1 >Reporter: Manoj Govindassamy >Assignee: Manoj Govindassamy > Fix For: 3.0.0-alpha4 > > Attachments: HADOOP-14223.01.patch, HADOOP-14223.02.patch > > > HDFS-6843 and HADOOP-13715 have enhanced {{FileStatus}} to include details on > whether the underlying path is Encrypted and Erasure Coded. The additional > details are embedded in the FsPermission high order bits. It would be really > helpful for debugging if FileStatus#toString() returns these new bits details > along with already existing one. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16112477#comment-16112477 ] Vishwajeet Dusane commented on HADOOP-14223: Raised HADOOP-14730 for tracking. > Extend FileStatus#toString() to include details like Erasure Coding and > Encryption > -- > > Key: HADOOP-14223 > URL: https://issues.apache.org/jira/browse/HADOOP-14223 > Project: Hadoop Common > Issue Type: Bug > Components: fs >Affects Versions: 3.0.0-alpha1 >Reporter: Manoj Govindassamy >Assignee: Manoj Govindassamy > Fix For: 3.0.0-alpha4 > > Attachments: HADOOP-14223.01.patch, HADOOP-14223.02.patch > > > HDFS-6843 and HADOOP-13715 have enhanced {{FileStatus}} to include details on > whether the underlying path is Encrypted and Erasure Coded. The additional > details are embedded in the FsPermission high order bits. It would be really > helpful for debugging if FileStatus#toString() returns these new bits details > along with already existing one. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16112461#comment-16112461 ] Vishwajeet Dusane commented on HADOOP-14223: 1. Test failure in [TestGetFileStatus.java#L68-L68|https://github.com/apache/hadoop/blob/4966a6e26e45d7dc36e0b270066ff7c87bcd00cc/hadoop-tools/hadoop-azure-datalake/src/test/java/org/apache/hadoop/fs/adl/TestGetFileStatus.java#L68-L68] is caused due to HDFS-6984 commit. [FileStatus.java#L104-L104|https://github.com/apache/hadoop/blob/12e44e7bdaf53d3720a89d32f0cc2717241bd6b2/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FileStatus.java#L104-L104]. Will raise a separate JIRA for the same. 2. Test failure in [TestGetFileStatus.java#L88-L89|https://github.com/apache/hadoop/blob/4966a6e26e45d7dc36e0b270066ff7c87bcd00cc/hadoop-tools/hadoop-azure-datalake/src/test/java/org/apache/hadoop/fs/adl/TestGetFileStatus.java#L88-L89], is also related to 1. Will raise a separate JIRA for the same. > Extend FileStatus#toString() to include details like Erasure Coding and > Encryption > -- > > Key: HADOOP-14223 > URL: https://issues.apache.org/jira/browse/HADOOP-14223 > Project: Hadoop Common > Issue Type: Bug > Components: fs >Affects Versions: 3.0.0-alpha1 >Reporter: Manoj Govindassamy >Assignee: Manoj Govindassamy > Fix For: 3.0.0-alpha4 > > Attachments: HADOOP-14223.01.patch, HADOOP-14223.02.patch > > > HDFS-6843 and HADOOP-13715 have enhanced {{FileStatus}} to include details on > whether the underlying path is Encrypted and Erasure Coded. The additional > details are embedded in the FsPermission high order bits. It would be really > helpful for debugging if FileStatus#toString() returns these new bits details > along with already existing one. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16112357#comment-16112357 ] Vishwajeet Dusane commented on HADOOP-14223: [~manojg], [~andrew.wang] and [~stevea] - hadoop azure datalake unit test are failing after this change. And [TestGetFileStatus.java#L68-L68|https://github.com/apache/hadoop/blob/4966a6e26e45d7dc36e0b270066ff7c87bcd00cc/hadoop-tools/hadoop-azure-datalake/src/test/java/org/apache/hadoop/fs/adl/TestGetFileStatus.java#L68-L68] {code:java} Assert.assertTrue(path + " should have Acl!", fileStatus.hasAcl()); {code} And [TestGetFileStatus.java#L88-L89|https://github.com/apache/hadoop/blob/4966a6e26e45d7dc36e0b270066ff7c87bcd00cc/hadoop-tools/hadoop-azure-datalake/src/test/java/org/apache/hadoop/fs/adl/TestGetFileStatus.java#L88-L89] {code:java} Assert.assertEquals(fileStatus.hasAcl(), fileStatus.getPermission().getAclBit()); {code} Looking ... > Extend FileStatus#toString() to include details like Erasure Coding and > Encryption > -- > > Key: HADOOP-14223 > URL: https://issues.apache.org/jira/browse/HADOOP-14223 > Project: Hadoop Common > Issue Type: Bug > Components: fs >Affects Versions: 3.0.0-alpha1 >Reporter: Manoj Govindassamy >Assignee: Manoj Govindassamy > Fix For: 3.0.0-alpha4 > > Attachments: HADOOP-14223.01.patch, HADOOP-14223.02.patch > > > HDFS-6843 and HADOOP-13715 have enhanced {{FileStatus}} to include details on > whether the underlying path is Encrypted and Erasure Coded. The additional > details are embedded in the FsPermission high order bits. It would be really > helpful for debugging if FileStatus#toString() returns these new bits details > along with already existing one. -- This message was sent by Atlassian JIRA (v6.4.14#64029) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15947978#comment-15947978 ] Manoj Govindassamy commented on HADOOP-14223: - Thanks for the review and commit help [~andrew.wang], [~ste...@apache.org]. > Extend FileStatus#toString() to include details like Erasure Coding and > Encryption > -- > > Key: HADOOP-14223 > URL: https://issues.apache.org/jira/browse/HADOOP-14223 > Project: Hadoop Common > Issue Type: Bug > Components: fs >Affects Versions: 3.0.0-alpha1 >Reporter: Manoj Govindassamy >Assignee: Manoj Govindassamy > Fix For: 3.0.0-alpha3 > > Attachments: HADOOP-14223.01.patch, HADOOP-14223.02.patch > > > HDFS-6843 and HADOOP-13715 have enhanced {{FileStatus}} to include details on > whether the underlying path is Encrypted and Erasure Coded. The additional > details are embedded in the FsPermission high order bits. It would be really > helpful for debugging if FileStatus#toString() returns these new bits details > along with already existing one. -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15947976#comment-15947976 ] Hudson commented on HADOOP-14223: - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #11492 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/11492/]) HADOOP-14223. Extend FileStatus#toString() to include details like (wang: rev 4966a6e26e45d7dc36e0b270066ff7c87bcd00cc) * (edit) hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestFileStatus.java * (edit) hadoop-common-project/hadoop-common/src/site/markdown/filesystem/filesystem.md * (edit) hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/viewfs/TestViewfsFileStatus.java * (edit) hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/TestFileStatus.java * (edit) hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestFileStatusWithECPolicy.java * (edit) hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/server/namenode/FSAclBaseTest.java * (edit) hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestWebHDFS.java * (edit) hadoop-tools/hadoop-azure-datalake/src/test/java/org/apache/hadoop/fs/adl/TestGetFileStatus.java * (edit) hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FileStatus.java * (edit) hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/TestS3AGetFileStatus.java > Extend FileStatus#toString() to include details like Erasure Coding and > Encryption > -- > > Key: HADOOP-14223 > URL: https://issues.apache.org/jira/browse/HADOOP-14223 > Project: Hadoop Common > Issue Type: Bug > Components: fs >Affects Versions: 3.0.0-alpha1 >Reporter: Manoj Govindassamy >Assignee: Manoj Govindassamy > Fix For: 3.0.0-alpha3 > > Attachments: HADOOP-14223.01.patch, HADOOP-14223.02.patch > > > HDFS-6843 and HADOOP-13715 have enhanced {{FileStatus}} to include details on > whether the underlying path is Encrypted and Erasure Coded. The additional > details are embedded in the FsPermission high order bits. It would be really > helpful for debugging if FileStatus#toString() returns these new bits details > along with already existing one. -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15946923#comment-15946923 ] Steve Loughran commented on HADOOP-14223: - LGTM +1 > Extend FileStatus#toString() to include details like Erasure Coding and > Encryption > -- > > Key: HADOOP-14223 > URL: https://issues.apache.org/jira/browse/HADOOP-14223 > Project: Hadoop Common > Issue Type: Bug > Components: fs >Affects Versions: 3.0.0-alpha1 >Reporter: Manoj Govindassamy >Assignee: Manoj Govindassamy > Attachments: HADOOP-14223.01.patch, HADOOP-14223.02.patch > > > HDFS-6843 and HADOOP-13715 have enhanced {{FileStatus}} to include details on > whether the underlying path is Encrypted and Erasure Coded. The additional > details are embedded in the FsPermission high order bits. It would be really > helpful for debugging if FileStatus#toString() returns these new bits details > along with already existing one. -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15946506#comment-15946506 ] Hadoop QA commented on HADOOP-14223: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 18s{color} | {color:blue} Docker mode activated. {color} | | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 8 new or modified test files. {color} | | {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 42s{color} | {color:blue} Maven dependency ordering for branch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 13m 21s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 18m 21s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 1m 59s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 2m 53s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 1m 21s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 4m 25s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 2m 23s{color} | {color:green} trunk passed {color} | | {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 17s{color} | {color:blue} Maven dependency ordering for patch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 2m 3s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 13m 37s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 13m 37s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 2m 4s{color} | {color:orange} root: The patch generated 2 new + 460 unchanged - 0 fixed = 462 total (was 460) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 3m 2s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 1m 33s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 5m 7s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 2m 36s{color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 8m 24s{color} | {color:red} hadoop-common in the patch failed. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 64m 34s{color} | {color:green} hadoop-hdfs in the patch passed. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 35s{color} | {color:green} hadoop-aws in the patch passed. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 3m 44s{color} | {color:green} hadoop-azure-datalake in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 42s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black}179m 14s{color} | {color:black} {color} | \\ \\ || Reason || Tests || | Failed junit tests | hadoop.ha.TestZKFailoverController | | | hadoop.security.TestKDiag | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:a9ad5d6 | | JIRA Issue | HADOOP-14223 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12860974/HADOOP-14223.02.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux a408d4ec9c24 3.13.0-106-generic #153-Ubuntu SMP Tue Dec 6 15:44:32 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / 063b513 | | Default Java | 1.8.0_121 | | findbugs | v3.0.0 | | checkstyle |
[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15946358#comment-15946358 ] Andrew Wang commented on HADOOP-14223: -- Thanks Manoj, I'm +1. I'll give it overnight so Steve can take a look too if he wants, though I believe that his review comments have been addressed. > Extend FileStatus#toString() to include details like Erasure Coding and > Encryption > -- > > Key: HADOOP-14223 > URL: https://issues.apache.org/jira/browse/HADOOP-14223 > Project: Hadoop Common > Issue Type: Bug > Components: fs >Affects Versions: 3.0.0-alpha1 >Reporter: Manoj Govindassamy >Assignee: Manoj Govindassamy > Attachments: HADOOP-14223.01.patch, HADOOP-14223.02.patch > > > HDFS-6843 and HADOOP-13715 have enhanced {{FileStatus}} to include details on > whether the underlying path is Encrypted and Erasure Coded. The additional > details are embedded in the FsPermission high order bits. It would be really > helpful for debugging if FileStatus#toString() returns these new bits details > along with already existing one. -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15944930#comment-15944930 ] Steve Loughran commented on HADOOP-14223: - looks good the specification in {{filesystem.md}} could be simplified {code} stat.hasAcl = hasACL(FS, p) {code} same can be done for the toString itself; no need for the if/else clauses {code} expected.append("hasAcl=").append(fileStatus.hasAcl()).append("; "); {code} + same for isEncrypted. Not only this terser code, there's no risk of branch misprediction in the CPU, hence a more efficient codepath. I am really impressed you actually wrote tests for the toString value; I wouldn't have expected that. Given the tests exist, their assert messages should include the toString value, so we can see what the string value is on a failing assertion. > Extend FileStatus#toString() to include details like Erasure Coding and > Encryption > -- > > Key: HADOOP-14223 > URL: https://issues.apache.org/jira/browse/HADOOP-14223 > Project: Hadoop Common > Issue Type: Bug > Components: fs >Affects Versions: 3.0.0-alpha1 >Reporter: Manoj Govindassamy >Assignee: Manoj Govindassamy > Attachments: HADOOP-14223.01.patch > > > HDFS-6843 and HADOOP-13715 have enhanced {{FileStatus}} to include details on > whether the underlying path is Encrypted and Erasure Coded. The additional > details are embedded in the FsPermission high order bits. It would be really > helpful for debugging if FileStatus#toString() returns these new bits details > along with already existing one. -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15944182#comment-15944182 ] Hadoop QA commented on HADOOP-14223: | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 5m 27s{color} | {color:blue} Docker mode activated. {color} | | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 0s{color} | {color:green} The patch does not contain any @author tags. {color} | | {color:green}+1{color} | {color:green} test4tests {color} | {color:green} 0m 0s{color} | {color:green} The patch appears to include 4 new or modified test files. {color} | | {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 2m 17s{color} | {color:blue} Maven dependency ordering for branch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 13m 45s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 16m 39s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 2m 0s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 2m 25s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 1m 4s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 3m 55s{color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 2m 2s{color} | {color:green} trunk passed {color} | | {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue} 0m 16s{color} | {color:blue} Maven dependency ordering for patch {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 1m 44s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 14m 53s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 14m 53s{color} | {color:green} the patch passed {color} | | {color:orange}-0{color} | {color:orange} checkstyle {color} | {color:orange} 2m 3s{color} | {color:orange} root: The patch generated 2 new + 414 unchanged - 0 fixed = 416 total (was 414) {color} | | {color:green}+1{color} | {color:green} mvnsite {color} | {color:green} 2m 32s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 1m 9s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s{color} | {color:green} The patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 4m 33s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 2m 21s{color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 10m 17s{color} | {color:green} hadoop-common in the patch passed. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 93m 48s{color} | {color:green} hadoop-hdfs in the patch passed. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 3m 44s{color} | {color:green} hadoop-azure-datalake in the patch passed. {color} | | {color:green}+1{color} | {color:green} asflicense {color} | {color:green} 0m 43s{color} | {color:green} The patch does not generate ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black}212m 20s{color} | {color:black} {color} | \\ \\ || Subsystem || Report/Notes || | Docker | Image:yetus/hadoop:a9ad5d6 | | JIRA Issue | HADOOP-14223 | | JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12860705/HADOOP-14223.01.patch | | Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle | | uname | Linux e248236094cf 3.13.0-107-generic #154-Ubuntu SMP Tue Dec 20 09:57:27 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux | | Build tool | maven | | Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh | | git revision | trunk / db2adf3 | | Default Java | 1.8.0_121 | | findbugs | v3.0.0 | | checkstyle | https://builds.apache.org/job/PreCommit-HADOOP-Build/11942/artifact/patchprocess/diff-checkstyle-root.txt | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/11942/testReport/ | | modules | C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs hadoop-tools/hadoop-azure-datalake U: . | | Console
[jira] [Commented] (HADOOP-14223) Extend FileStatus#toString() to include details like Erasure Coding and Encryption
[ https://issues.apache.org/jira/browse/HADOOP-14223?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15944000#comment-15944000 ] Andrew Wang commented on HADOOP-14223: -- Hi Manoj, thanks for working on this! Had a few review comments: * I like adding hasAcl to FileStatus, since an FsPermission seems to encapsulate just the Unix-style permissions. Seems like we have some duplication now with the FsPermission getAclBit / getEncryptedBit / getErausreCodedBit getters, perhaps we should deprecate them? getECBit we can also annotate as Private. I'm hoping we can clean up these fields/getters in FsPermission when HDFS-6984 goes in. * I don't think toString should be required as part of assertErasureCoded in ContractTestUtils, since as we discussed on HADOOP-13715, toString isn't part of the public contract for a HCFS. It's fine to have toString asserts in Hadoop or HDFS-specific tests though. * Nit in filesystem.md: A given path only has a single ACL composed of multiple AC entries, so I think "any ACLs" -> "an ACL". Another nit is that Encryption or Erasure Coded aren't proper nouns, so I'd prefer them to be lower case. {code:title=TestFileStatus} if (fileStatus.hasAcl()) { expected.append("hasAcl=").append(true).append("; "); } else { expected.append("hasAcl=").append(false).append("; "); } {code} Could these be made more concise, e.g. {code} expected.append("hasAcl=").append(fileStatus.hasAcl()).append("; "); {code} > Extend FileStatus#toString() to include details like Erasure Coding and > Encryption > -- > > Key: HADOOP-14223 > URL: https://issues.apache.org/jira/browse/HADOOP-14223 > Project: Hadoop Common > Issue Type: Bug > Components: fs >Affects Versions: 3.0.0-alpha1 >Reporter: Manoj Govindassamy >Assignee: Manoj Govindassamy > Attachments: HADOOP-14223.01.patch > > > HDFS-6843 and HADOOP-13715 have enhanced {{FileStatus}} to include details on > whether the underlying path is Encrypted and Erasure Coded. The additional > details are embedded in the FsPermission high order bits. It would be really > helpful for debugging if FileStatus#toString() returns these new bits details > along with already existing one. -- This message was sent by Atlassian JIRA (v6.3.15#6346) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org