[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14973992#comment-14973992 ] Hadoop QA commented on HADOOP-12040: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 8s {color} | {color:blue} docker + precommit patch detected. {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:green}+1{color} | {color:green} mvninstall {color} | {color:green} 3m 33s {color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 4m 23s {color} | {color:green} trunk passed with JDK v1.8.0_60 {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 4m 6s {color} | {color:green} trunk passed with JDK v1.7.0_79 {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 0m 57s {color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 40s {color} | {color:green} trunk passed {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 1m 54s {color} | {color:red} hadoop-hdfs-project/hadoop-hdfs in trunk cannot run convertXmlToText from findbugs {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 2m 22s {color} | {color:green} trunk passed with JDK v1.8.0_60 {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 3m 14s {color} | {color:green} trunk passed with JDK v1.7.0_79 {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 2m 40s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 4m 17s {color} | {color:green} the patch passed with JDK v1.8.0_60 {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 4m 17s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 4m 8s {color} | {color:green} the patch passed with JDK v1.7.0_79 {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 4m 8s {color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} checkstyle {color} | {color:red} 0m 55s {color} | {color:red} Patch generated 4 new checkstyle issues in root (total was 118, now 119). {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 38s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s {color} | {color:green} Patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 5m 46s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 2m 20s {color} | {color:green} the patch passed with JDK v1.8.0_60 {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 3m 10s {color} | {color:green} the patch passed with JDK v1.7.0_79 {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 6m 42s {color} | {color:green} hadoop-common in the patch passed with JDK v1.8.0_60. {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 53m 37s {color} | {color:red} hadoop-hdfs in the patch failed with JDK v1.8.0_60. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 27s {color} | {color:green} hadoop-hdfs-client in the patch passed with JDK v1.8.0_60. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 7m 9s {color} | {color:green} hadoop-common in the patch passed with JDK v1.7.0_79. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 50m 20s {color} | {color:green} hadoop-hdfs in the patch passed with JDK v1.7.0_79. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 26s {color} | {color:green} hadoop-hdfs-client in the patch passed with JDK v1.7.0_79. {color} | | {color:red}-1{color} | {color:red} asflicense {color} | {color:red} 0m 19s {color} | {color:red} Patch generated 56 ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 169m 8s {color} | {color:black} {color} | \\ \\ || Reason || Tests || | JDK v1.7.0_79 Failed junit tests | hadoop.hdfs.server.datanode.TestBlockRecovery | | | hadoop.hdfs.server.namenode.h
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14975457#comment-14975457 ] Kai Zheng commented on HADOOP-12040: [~hitliuyi] and [~zhz], would you help review this one? It is highly relied by ISA-L and new Java coders. Thanks. > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14975907#comment-14975907 ] Hadoop QA commented on HADOOP-12040: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 12s {color} | {color:blue} docker + precommit patch detected. {color} | | {color:green}+1{color} | {color:green} @author {color} | {color:green} 0m 1s {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:green}+1{color} | {color:green} mvninstall {color} | {color:green} 3m 55s {color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 5m 45s {color} | {color:green} trunk passed with JDK v1.8.0_60 {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 5m 8s {color} | {color:green} trunk passed with JDK v1.7.0_79 {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 1m 13s {color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 49s {color} | {color:green} trunk passed {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 2m 25s {color} | {color:red} hadoop-hdfs-project/hadoop-hdfs in trunk cannot run convertXmlToText from findbugs {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 3m 7s {color} | {color:green} trunk passed with JDK v1.8.0_60 {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 4m 9s {color} | {color:green} trunk passed with JDK v1.7.0_79 {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 3m 10s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 5m 41s {color} | {color:green} the patch passed with JDK v1.8.0_60 {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 5m 41s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 5m 18s {color} | {color:green} the patch passed with JDK v1.7.0_79 {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 5m 18s {color} | {color:green} the patch passed {color} | | {color:red}-1{color} | {color:red} checkstyle {color} | {color:red} 1m 14s {color} | {color:red} Patch generated 2 new checkstyle issues in root (total was 118, now 117). {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 49s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s {color} | {color:green} Patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 7m 30s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 3m 20s {color} | {color:green} the patch passed with JDK v1.8.0_60 {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 4m 14s {color} | {color:green} the patch passed with JDK v1.7.0_79 {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 8m 43s {color} | {color:red} hadoop-common in the patch failed with JDK v1.8.0_60. {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 67m 41s {color} | {color:red} hadoop-hdfs in the patch failed with JDK v1.8.0_60. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 31s {color} | {color:green} hadoop-hdfs-client in the patch passed with JDK v1.8.0_60. {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 7m 47s {color} | {color:red} hadoop-common in the patch failed with JDK v1.7.0_79. {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 64m 21s {color} | {color:red} hadoop-hdfs in the patch failed with JDK v1.7.0_79. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 36s {color} | {color:green} hadoop-hdfs-client in the patch passed with JDK v1.7.0_79. {color} | | {color:red}-1{color} | {color:red} asflicense {color} | {color:red} 0m 27s {color} | {color:red} Patch generated 56 ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 214m 33s {color} | {color:black} {color} | \\ \\ || Reason || Tests || | JDK v1.7.0_79 Failed junit tests | hadoop.security.TestShellBasedIdMapping | | | hadoop.ipc.TestIPC | | | hadoop.test.TestTimedOutTestsList
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14976024#comment-14976024 ] Yi Liu commented on HADOOP-12040: - Generally looks good, Kai. 1. You need to cleanup the checkstype issue. For example, some line is longer than 80 characters. 2. Some related tests show failure, such as TestRecoverStripedFile 3. {code}+for (int i = 0; i < erasedIndexes.length; i++) { + if (erasedIndexes[i] >= getNumDataUnits()) { +erasedIndexes2[idx++] = erasedIndexes[i] - getNumDataUnits(); +numErasedParityUnits++; + } +} +for (int i = 0; i < erasedIndexes.length; i++) { + if (erasedIndexes[i] < getNumDataUnits()) { +erasedIndexes2[idx++] = erasedIndexes[i] + getNumParityUnits(); +numErasedDataUnits++; + } +} {code} This can be done in a {{for}}. > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14976060#comment-14976060 ] Walter Su commented on HADOOP-12040: Thanks [~drankye]! The v3 patch LGTM. (Non-binding) > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14976073#comment-14976073 ] Walter Su commented on HADOOP-12040: Thanks [~hitliuyi] for pointing out the style issues. BTW, I think the failure of {{TestRecoverStripedFile}} is not related to this. With the patch It passes locally. The occasional failure could have the same reason as HDFS-9275. > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14976299#comment-14976299 ] Kai Zheng commented on HADOOP-12040: Thanks Yi and Walter for the review and comments! I will double check the mentioned issues. > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14977510#comment-14977510 ] Yi Liu commented on HADOOP-12040: - For the comment #3, it's not convenient to do, so don't need to address it. > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14977564#comment-14977564 ] Yi Liu commented on HADOOP-12040: - +1 pending Jenkins. > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch, HADOOP-12040-v4.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14977901#comment-14977901 ] Hadoop QA commented on HADOOP-12040: | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:blue}0{color} | {color:blue} reexec {color} | {color:blue} 0m 14s {color} | {color:blue} docker + precommit patch detected. {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:green}+1{color} | {color:green} mvninstall {color} | {color:green} 3m 11s {color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 4m 47s {color} | {color:green} trunk passed with JDK v1.8.0_66 {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 4m 32s {color} | {color:green} trunk passed with JDK v1.7.0_79 {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 1m 0s {color} | {color:green} trunk passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 42s {color} | {color:green} trunk passed {color} | | {color:red}-1{color} | {color:red} findbugs {color} | {color:red} 2m 1s {color} | {color:red} hadoop-hdfs-project/hadoop-hdfs in trunk cannot run convertXmlToText from findbugs {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 2m 31s {color} | {color:green} trunk passed with JDK v1.8.0_66 {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 3m 32s {color} | {color:green} trunk passed with JDK v1.7.0_79 {color} | | {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 2m 44s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 4m 41s {color} | {color:green} the patch passed with JDK v1.8.0_66 {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 4m 41s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} compile {color} | {color:green} 4m 29s {color} | {color:green} the patch passed with JDK v1.7.0_79 {color} | | {color:green}+1{color} | {color:green} javac {color} | {color:green} 4m 29s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 1m 3s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green} 0m 43s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} whitespace {color} | {color:green} 0m 0s {color} | {color:green} Patch has no whitespace issues. {color} | | {color:green}+1{color} | {color:green} findbugs {color} | {color:green} 6m 21s {color} | {color:green} the patch passed {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 2m 37s {color} | {color:green} the patch passed with JDK v1.8.0_66 {color} | | {color:green}+1{color} | {color:green} javadoc {color} | {color:green} 3m 32s {color} | {color:green} the patch passed with JDK v1.7.0_79 {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 7m 59s {color} | {color:red} hadoop-common in the patch failed with JDK v1.8.0_66. {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 68m 19s {color} | {color:red} hadoop-hdfs in the patch failed with JDK v1.8.0_66. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 29s {color} | {color:green} hadoop-hdfs-client in the patch passed with JDK v1.8.0_66. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 8m 33s {color} | {color:green} hadoop-common in the patch passed with JDK v1.7.0_79. {color} | | {color:red}-1{color} | {color:red} unit {color} | {color:red} 68m 50s {color} | {color:red} hadoop-hdfs in the patch failed with JDK v1.7.0_79. {color} | | {color:green}+1{color} | {color:green} unit {color} | {color:green} 0m 29s {color} | {color:green} hadoop-hdfs-client in the patch passed with JDK v1.7.0_79. {color} | | {color:red}-1{color} | {color:red} asflicense {color} | {color:red} 0m 20s {color} | {color:red} Patch generated 56 ASF License warnings. {color} | | {color:black}{color} | {color:black} {color} | {color:black} 208m 55s {color} | {color:black} {color} | \\ \\ || Reason || Tests || | JDK v1.7.0_79 Failed junit tests | hadoop.metrics2.impl.TestMetricsSystemImpl | | | hadoop.hdfs.server.namenode.TestCheckpoint | | | hadoop.hdfs.TestBlockReaderLocal | | | hadoo
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14977915#comment-14977915 ] Yi Liu commented on HADOOP-12040: - Will commit shortly > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch, HADOOP-12040-v4.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14978142#comment-14978142 ] Hudson commented on HADOOP-12040: - FAILURE: Integrated in Hadoop-Yarn-trunk #1329 (See [https://builds.apache.org/job/Hadoop-Yarn-trunk/1329/]) HADOOP-12040. Adjust inputs order for the decode API in raw erasure (yliu: rev c201cf951d5adefefe7c68e882a0c07962248577) * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSStripedInputStream.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/TestCoderBase.java * hadoop-common-project/hadoop-common/CHANGES.txt * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/AbstractRawErasureCoder.java * hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSStripedInputStream.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestErasureCoderBase.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RawErasureDecoder.java * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/util/StripedBlockUtil.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestRSErasureCoder.java * hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/erasurecode/ErasureCodingWorker.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/coder/AbstractErasureDecoder.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RSRawDecoder.java > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Fix For: 2.8.0 > > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch, HADOOP-12040-v4.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14978244#comment-14978244 ] Hudson commented on HADOOP-12040: - FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #593 (See [https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/593/]) HADOOP-12040. Adjust inputs order for the decode API in raw erasure (yliu: rev c201cf951d5adefefe7c68e882a0c07962248577) * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/coder/AbstractErasureDecoder.java * hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSStripedInputStream.java * hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/erasurecode/ErasureCodingWorker.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestErasureCoderBase.java * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/util/StripedBlockUtil.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/TestCoderBase.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/AbstractRawErasureCoder.java * hadoop-common-project/hadoop-common/CHANGES.txt * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSStripedInputStream.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestRSErasureCoder.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RawErasureDecoder.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RSRawDecoder.java > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Fix For: 2.8.0 > > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch, HADOOP-12040-v4.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14978311#comment-14978311 ] Hudson commented on HADOOP-12040: - FAILURE: Integrated in Hadoop-trunk-Commit #8718 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/8718/]) HADOOP-12040. Adjust inputs order for the decode API in raw erasure (yliu: rev c201cf951d5adefefe7c68e882a0c07962248577) * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/util/StripedBlockUtil.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/AbstractRawErasureCoder.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestErasureCoderBase.java * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSStripedInputStream.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RawErasureDecoder.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/coder/AbstractErasureDecoder.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestRSErasureCoder.java * hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/erasurecode/ErasureCodingWorker.java * hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSStripedInputStream.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RSRawDecoder.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/TestCoderBase.java * hadoop-common-project/hadoop-common/CHANGES.txt > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Fix For: 2.8.0 > > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch, HADOOP-12040-v4.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14978334#comment-14978334 ] Hudson commented on HADOOP-12040: - FAILURE: Integrated in Hadoop-Hdfs-trunk #2483 (See [https://builds.apache.org/job/Hadoop-Hdfs-trunk/2483/]) HADOOP-12040. Adjust inputs order for the decode API in raw erasure (yliu: rev c201cf951d5adefefe7c68e882a0c07962248577) * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSStripedInputStream.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RawErasureDecoder.java * hadoop-common-project/hadoop-common/CHANGES.txt * hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/erasurecode/ErasureCodingWorker.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestRSErasureCoder.java * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/util/StripedBlockUtil.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/TestCoderBase.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RSRawDecoder.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/AbstractRawErasureCoder.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/coder/AbstractErasureDecoder.java * hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSStripedInputStream.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestErasureCoderBase.java > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Fix For: 2.8.0 > > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch, HADOOP-12040-v4.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14978396#comment-14978396 ] Hudson commented on HADOOP-12040: - FAILURE: Integrated in Hadoop-Mapreduce-trunk #2537 (See [https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2537/]) HADOOP-12040. Adjust inputs order for the decode API in raw erasure (yliu: rev c201cf951d5adefefe7c68e882a0c07962248577) * hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/erasurecode/ErasureCodingWorker.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RSRawDecoder.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/coder/AbstractErasureDecoder.java * hadoop-common-project/hadoop-common/CHANGES.txt * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/AbstractRawErasureCoder.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestErasureCoderBase.java * hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSStripedInputStream.java * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSStripedInputStream.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RawErasureDecoder.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestRSErasureCoder.java * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/util/StripedBlockUtil.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/TestCoderBase.java > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Fix For: 2.8.0 > > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch, HADOOP-12040-v4.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14978480#comment-14978480 ] Hudson commented on HADOOP-12040: - SUCCESS: Integrated in Hadoop-Yarn-trunk-Java8 #607 (See [https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/607/]) HADOOP-12040. Adjust inputs order for the decode API in raw erasure (yliu: rev c201cf951d5adefefe7c68e882a0c07962248577) * hadoop-common-project/hadoop-common/CHANGES.txt * hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSStripedInputStream.java * hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/erasurecode/ErasureCodingWorker.java * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/util/StripedBlockUtil.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RawErasureDecoder.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/AbstractRawErasureCoder.java * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSStripedInputStream.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RSRawDecoder.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/coder/AbstractErasureDecoder.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestErasureCoderBase.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/TestCoderBase.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestRSErasureCoder.java > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Fix For: 2.8.0 > > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch, HADOOP-12040-v4.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14979055#comment-14979055 ] Hudson commented on HADOOP-12040: - FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #546 (See [https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/546/]) HADOOP-12040. Adjust inputs order for the decode API in raw erasure (yliu: rev c201cf951d5adefefe7c68e882a0c07962248577) * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/AbstractRawErasureCoder.java * hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/TestDFSStripedInputStream.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RawErasureDecoder.java * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/DFSStripedInputStream.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestRSErasureCoder.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/coder/TestErasureCoderBase.java * hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/util/StripedBlockUtil.java * hadoop-common-project/hadoop-common/CHANGES.txt * hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/datanode/erasurecode/ErasureCodingWorker.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/coder/AbstractErasureDecoder.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/erasurecode/TestCoderBase.java * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/RSRawDecoder.java > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Fix For: 2.8.0 > > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch, HADOOP-12040-v4.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15799507#comment-15799507 ] Junping Du commented on HADOOP-12040: - I believe this patch went only to trunk. Remove 2.8 from fix version. > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Fix For: 3.0.0-alpha1 > > Attachments: HADOOP-12040-HDFS-7285-v1.patch, HADOOP-12040-v2.patch, > HADOOP-12040-v3.patch, HADOOP-12040-v4.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332) - To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14575284#comment-14575284 ] Zhe Zhang commented on HADOOP-12040: Thanks for taking this task Kai! It's a good improvement for robust and readable code. > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (HADOOP-12040) Adjust inputs order for the decode API in raw erasure coder
[ https://issues.apache.org/jira/browse/HADOOP-12040?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14699453#comment-14699453 ] Hadoop QA commented on HADOOP-12040: \\ \\ | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem || Runtime || Comment || | {color:red}-1{color} | pre-patch | 16m 38s | Findbugs (version ) appears to be broken on HDFS-7285. | | {color:green}+1{color} | @author | 0m 0s | The patch does not contain any @author tags. | | {color:green}+1{color} | tests included | 0m 0s | The patch appears to include 4 new or modified test files. | | {color:green}+1{color} | javac | 8m 8s | There were no new javac warning messages. | | {color:green}+1{color} | javadoc | 10m 30s | There were no new javadoc warning messages. | | {color:red}-1{color} | release audit | 0m 15s | The applied patch generated 1 release audit warnings. | | {color:green}+1{color} | checkstyle | 1m 23s | There were no new checkstyle issues. | | {color:red}-1{color} | whitespace | 0m 1s | The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix. | | {color:green}+1{color} | install | 1m 45s | mvn install still works. | | {color:green}+1{color} | eclipse:eclipse | 0m 32s | The patch built with eclipse:eclipse. | | {color:red}-1{color} | findbugs | 5m 36s | The patch appears to introduce 5 new Findbugs (version 3.0.0) warnings. | | {color:red}-1{color} | common tests | 22m 3s | Tests failed in hadoop-common. | | {color:red}-1{color} | hdfs tests | 197m 18s | Tests failed in hadoop-hdfs. | | | | 264m 14s | | \\ \\ || Reason || Tests || | FindBugs | module:hadoop-hdfs | | Failed unit tests | hadoop.net.TestNetUtils | | | hadoop.ha.TestZKFailoverController | | | hadoop.fs.TestUrlStreamHandler | | | hadoop.fs.TestFcHdfsSetUMask | | | hadoop.hdfs.server.namenode.TestFileTruncate | | | hadoop.fs.contract.hdfs.TestHDFSContractAppend | | | hadoop.fs.TestSymlinkHdfsFileSystem | | | hadoop.hdfs.TestClientProtocolForPipelineRecovery | | | hadoop.hdfs.TestFSInputChecker | | | hadoop.tracing.TestTracing | | | hadoop.tracing.TestTracingShortCircuitLocalRead | | | hadoop.hdfs.TestWriteStripedFileWithFailure | | | hadoop.fs.loadGenerator.TestLoadGenerator | | | hadoop.fs.TestGlobPaths | | | hadoop.hdfs.TestReplaceDatanodeOnFailure | | | hadoop.fs.TestSymlinkHdfsDisable | | Timed out tests | org.apache.hadoop.hdfs.server.mover.TestMover | | | org.apache.hadoop.cli.TestHDFSCLI | \\ \\ || Subsystem || Report/Notes || | Patch URL | http://issues.apache.org/jira/secure/attachment/12750763/HADOOP-12040-HDFS-7285-v1.patch | | Optional Tests | javadoc javac unit findbugs checkstyle | | git revision | HDFS-7285 / b57c9a3 | | Release Audit | https://builds.apache.org/job/PreCommit-HADOOP-Build/7490/artifact/patchprocess/patchReleaseAuditProblems.txt | | whitespace | https://builds.apache.org/job/PreCommit-HADOOP-Build/7490/artifact/patchprocess/whitespace.txt | | Findbugs warnings | https://builds.apache.org/job/PreCommit-HADOOP-Build/7490/artifact/patchprocess/newPatchFindbugsWarningshadoop-hdfs.html | | hadoop-common test log | https://builds.apache.org/job/PreCommit-HADOOP-Build/7490/artifact/patchprocess/testrun_hadoop-common.txt | | hadoop-hdfs test log | https://builds.apache.org/job/PreCommit-HADOOP-Build/7490/artifact/patchprocess/testrun_hadoop-hdfs.txt | | Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/7490/testReport/ | | Java | 1.7.0_55 | | uname | Linux asf904.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux | | Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/7490/console | This message was automatically generated. > Adjust inputs order for the decode API in raw erasure coder > --- > > Key: HADOOP-12040 > URL: https://issues.apache.org/jira/browse/HADOOP-12040 > Project: Hadoop Common > Issue Type: Sub-task >Reporter: Kai Zheng >Assignee: Kai Zheng > Fix For: HDFS-7285 > > Attachments: HADOOP-12040-HDFS-7285-v1.patch > > > Currently we used the parity units + data units order for the inputs, > erasedIndexes and outputs parameters in the decode call in raw erasure coder, > which inherited from HDFS-RAID due to impact enforced by {{GaliosField}}. As > [~zhz] pointed and [~hitliuyi] felt, we'd better change the order to make it > natural for HDFS usage, where usually data blocks are before parity blocks in > a group. Doing this would avoid some reordering tricky logic. -- This message was sent by Atlassian JIRA (v6.3.4#6332)