[GitHub] [ozone] swagle edited a comment on pull request #1973: HDDS-4882. Introduce the ReplicationConfig and modify the proto files

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

swagle edited a comment on pull request #1973:
URL: https://github.com/apache/ozone/pull/1973#issuecomment-806933861


   > We discussed this question with @umamaheswararao and @sodonnel on the EC sync, and based on the feedback I would like to highlight my previous comment.
   > 
   > > The first part can be done easily with introducing new Java classes (and only one ECReplicationConfig in proto files):
   > > This change is separated and uploaded in the PR #2068
   > 
   > We can do it in a more lightweight and more compatible way (use ReplicationConfig just in the Java code and for new replication types in protos). This is uploaded in a separated PR (please check it). This PR talks about changing the old replication types which is more like a strategic question...
   
   @elek Is 2068 for the master and this PR for the branch (changes to be brought in via upgrades)? Do we have it the other way around?


-- 
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:
users@infra.apache.org



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