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


   Adding a few more columns that I think we need to define:
   
   Case | Client version (A) | Server version (B) | Wire message (C) | Message 
stored in OM Raft log and DB (D) | Behavior before finalize (E) | Behavior 
after finalize (F) | Downgrade result (G)
   -- | -- | -- | -- | -- | -- | -- | -- 
   1 | New | Old |  uses factor/type AND ReplicationConfig (first can be 
optional if server is known to be new) | store factor/type   | | | server is 
old, no downgrade
   2 | Old | Old | uses factor/type |  store factor/type | Business as usual |  
Business as usual | server is old, no downgrade
   3 | New | New | uses factor/type AND ReplicationConfig  |  store factor/type 
+ ReplicationConfig | | | can be done until we store old factor/type everywhere 
(finalize)   
   4 | Old | New | uses factor/type | store factor/type + ReplicationConfig  | 
| | factor/type is used to calculate ReplicationConfig, both can be stored
   
   G3 and G4 also need updates.
   


-- 
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]

Reply via email to