Changes between Version 1 and Version 2 of Ticket #1670, comment 41


Ignore:
Timestamp:
2013-07-08T00:38:11Z (11 years ago)
Author:
daira
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #1670, comment 41

    v1 v2  
    22> It looks like the mutable filenode modification code in {{{MutableFileVersion}}} reacts to an {{{UncoordinatedWriteError}}} by performing a map update, then trying the operation again. That map update will update the {{{MutableFileVersion}}}'s internal servermap, but not its internal verinfo tuple. If the {{{UncoordinatedWriteError}}} is due to a concurrent update operation that updated all or most (enough for the version to be unrecoverable, anyway) of the shares associated with the {{{MutableFileVersion}}}'s version of the mutable file, then we would see the {{{KeyError}}} reported by vikarti and others.
    33
    4 So, isn't the correct fix for this, without making any design changes, just for {{{MutableFileVersion}}} to make sure that its verinfo type is also updated when it does a mapupdate in this case?
     4So, isn't the correct fix for this, without making any design changes, just for {{{MutableFileVersion}}} to make sure that its verinfo tuple is also updated when it does a mapupdate in this case?