[ https://issues.apache.org/jira/browse/HDFS-8123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14493742#comment-14493742 ]
Rakesh R commented on HDFS-8123: -------------------------------- Thanks [~drankye] for the comments. bq. For somehow feeling maybe we could use erasurecoding instead of erasurecode for the new proto file? How others would think? I agree with you, {{erasurecoding}} is better one. bq. It might not be good to have the following in the new proto file right now? I'm not sure. I think, its not required to add this section. I will remove this section. I copied the license and header from {{hdfs.proto}} > 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)