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

Kai Zheng commented on HADOOP-12924:
------------------------------------

Thanks Rui for the great update and nice tests! Two more comments:
* Maybe we can define RS_DEFAULT_CODEC_NAME and the like in an enum?
* Could we move the static schema definitions like RS_6_3_SCHEMA out of 
{{HdfsConstants}} maybe to {{CodecUtil}}? HdfsConstants isn't a good place for 
such.

> Add default coder key for creating raw coders
> ---------------------------------------------
>
>                 Key: HADOOP-12924
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12924
>             Project: Hadoop Common
>          Issue Type: Sub-task
>            Reporter: Rui Li
>            Assignee: Rui Li
>            Priority: Minor
>              Labels: hdfs-ec-3.0-must-do
>         Attachments: HADOOP-12924.1.patch, HADOOP-12924.2.patch, 
> HADOOP-12924.3.patch, HADOOP-12924.4.patch
>
>
> As suggested 
> [here|https://issues.apache.org/jira/browse/HADOOP-12826?focusedCommentId=15194402&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15194402].



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

Reply via email to