elek commented on pull request #920: URL: https://github.com/apache/hadoop-ozone/pull/920#issuecomment-630017548
Thanks to create this patch @maobaolong It seems to be related to the storage class proposal published on the mailing list: https://lists.apache.org/thread.html/r1e2a5d5581abe9dd09834305ca65a6807f37bd229a07b8b31bda32ad%40%3Cozone-dev.hadoop.apache.org%3E With storage class you can remove any replication related information from the proto and you don't need any number just the information about the storage class. One storage class can be resolved as (RATIS/THREE --> CLOSED/TWO) but you can also support (RATIS/ONE --> CLOSED/TWO). I would prefer to follow (or at least discuss) this more generic approach. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
