[GitHub] [ozone] fapifta commented on pull request #3006: HDDS-6172: EC: Document the Ozone EC

[GitBox <gi...@apache.org>]

fapifta commented on pull request #3006:
URL: https://github.com/apache/ozone/pull/3006#issuecomment-1020685472


   Hi @umamaheswararao,
   
   thank you for writing the documentation parts for the EC feature. I have added a couple of inline comments mainly for spelling issues, or where I have not understood well the sentence for the first read. (It might be because of my non-native english  skills, so I might not be right everywhere).
   
   In general I would like to ask you to proof read the text one more time and please take care of some inconsistencies in writing different names. What I found inconsistent is the mixing of lower/uppercase forms like:
   ec vs Ec vs EC
   erasure coding vs Erasure coding vs Erasure Coding
   replication config vs Replication config vs Replication configuration mixed with an ec prefix sometimes
   
   At some points while I was reading I really missed an article in front of some words, and sometimes I felt the one I see is not really necessary. Again this can be my non-nativeness, and you might be perfectly right with the usage or lack of the article, hence when you read the text again, please consider this, and if articles are really missed or if they are not needed then please fix it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org