Re: Review Request 58599: GEODE:2802 Tombstone version vector to contain only the members that generate the tombstone.

2017-04-25 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58599/ --- (Updated April 26, 2017, 1:29 a.m.) Review request for geode, Darrel

Re: Review Request 58599: GEODE:2802 Tombstone version vector to contain only the members that generate the tombstone.

2017-04-25 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58599/#review172974 --- Ship it! Ship It! - Darrel Schneider On April 24, 2017,

Re: Review Request 58599: GEODE:2802 Tombstone version vector to contain only the members that generate the tombstone.

2017-04-24 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58599/ --- (Updated April 25, 2017, 1:46 a.m.) Review request for geode, Darrel

Re: Review Request 58599: GEODE:2802 Tombstone version vector to contain only the members that generate the tombstone.

2017-04-24 Thread anilkumar gingade
> On April 21, 2017, 5:15 p.m., Darrel Schneider wrote: > > This fix does address one particular failure. > > Should you also make some other changes to prevent and/or detect when it > > happens in other contexts? > > One issue is the serialization of the TombstoneMessage. I think the toData >

Re: Review Request 58599: GEODE:2802 Tombstone version vector to contain only the members that generate the tombstone.

2017-04-21 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58599/#review172678 --- This fix does address one particular failure. Should you also

Review Request 58599: GEODE:2802 Tombstone version vector to contain only the members that generate the tombstone.

2017-04-20 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58599/ --- Review request for geode, Darrel Schneider, Eric Shu, and Lynn Gallinat.