xiaoyuyao edited a comment on pull request #920: URL: https://github.com/apache/hadoop-ozone/pull/920#issuecomment-629675867
Thanks for the document, that helps. Instead of changing the Enum to int, have you consider adding the additional factors into existing Enum (e.g. odd number < 32) ? Here are some of my thoughts 1. We won't be able to support large replication factor or arbitrary replication factor (even) in near future due to raft limitations. 2. We will have better control over the replication factor that we can support. 3. We will need to maintain backward and forward compatibility for future changes. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org