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

Xinwei Qin  commented on HDFS-8156:
-----------------------------------

A minor comment:
The {{toString()}} method will output string similar to the following format:
{code}
ECSchema=[Name=xxx,option1=xxx,option2=xxx,option3=xxx,]
{code}
The last option will have a comma. Remove the last comma and add a whitespace 
between every two options will be better.

> Add/implement necessary APIs even we just have the system default schema
> ------------------------------------------------------------------------
>
>                 Key: HDFS-8156
>                 URL: https://issues.apache.org/jira/browse/HDFS-8156
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Kai Zheng
>            Assignee: Kai Zheng
>         Attachments: HDFS-8156-v1.patch, HDFS-8156-v2.patch, 
> HDFS-8156-v3.patch, HDFS-8156-v4.patch, HDFS-8156-v5.patch, 
> HDFS-8156-v6.patch, HDFS-8156-v7.patch, HDFS-8156-v8.patch
>
>
> According to the discussion here, this issue was repurposed and modified.
> This is to add and implement some necessary APIs even we just have the system 
> default schema, to resolve some TODOs left for HDFS-7859 and HDFS-7866 as 
> they're still subject to further discussion.



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

Reply via email to