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

Kai Zheng commented on HDFS-8123:
---------------------------------

1. For somehow feeling maybe we could use {{erasurecoding}} instead of 
{{erasurecode}} for the new proto file? How others would think?
2. It might not be good to have the following in the new proto file right now? 
I'm not sure. 
{code}
These .proto interfaces are private and stable.
{code}

[~vinayrpet], I'm not familiar with this aspect. Would you take a look at this 
and give your comments? Thanks.

> Erasure Coding: Better to move EC related proto messages to a separate 
> 'erasurecode.proto' file
> -----------------------------------------------------------------------------------------------
>
>                 Key: HDFS-8123
>                 URL: https://issues.apache.org/jira/browse/HDFS-8123
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Rakesh R
>            Assignee: Rakesh R
>         Attachments: HDFS-8123-001.patch, HDFS-8123-002.patch, 
> HDFS-8123-003.patch
>
>
> While reviewing the code I've noticed EC related proto messages are getting 
> added into {{hdfs.proto}}. IMHO, for better maintainability of the erasure 
> code feature, its good to move this to a separate {{erasurecode.proto}} file.



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

Reply via email to