You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2015/09/30 17:59:28 UTC

[jira] [Commented] (HADOOP-11818) Minor improvements for erasurecode classes

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

Hudson commented on HADOOP-11818:
---------------------------------

FAILURE: Integrated in Hadoop-Hdfs-trunk #2379 (See [https://builds.apache.org/job/Hadoop-Hdfs-trunk/2379/])
HADOOP-11818 Minor improvements for erasurecode classes. Contributed by (zhezhang: rev 971bd5e885ee1528fd0e7e9486e3b54cde0d02e2)
* hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/coder/RSErasureDecoder.java
* hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/rawcoder/util/RSUtil.java
* hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/SchemaLoader.java
* hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/coder/XORErasureDecoder.java
* hadoop-common-project/hadoop-common/CHANGES-HDFS-EC-7285.txt
* hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/erasurecode/coder/RSErasureEncoder.java


> Minor improvements for erasurecode classes
> ------------------------------------------
>
>                 Key: HADOOP-11818
>                 URL: https://issues.apache.org/jira/browse/HADOOP-11818
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: io
>            Reporter: Rakesh R
>            Assignee: Rakesh R
>             Fix For: HDFS-7285
>
>         Attachments: HADOOP-11818-001.patch, HADOOP-11818-002.patch
>
>
> Following are few observations while going through the {{erasurecode}} classes.
> \\
> - Use {{org.slf4j.Logger}} instead of {{org.apache.commons.logging.Log}}
> - {{RSErasureDecoder}}, {{RSErasureEncoder}}, {{RSUtil}} missing Apache License header.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)