----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51510/#review147364 -----------------------------------------------------------
Ship it! Ship It! - Darrel Schneider On Aug. 30, 2016, 1:26 p.m., Dan Smith wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51510/ > ----------------------------------------------------------- > > (Updated Aug. 30, 2016, 1:26 p.m.) > > > Review request for geode, Bruce Schuchardt and Darrel Schneider. > > > Repository: geode > > > Description > ------- > > In LocalRegion.basicPutPart2, check for a TOMBSTONE using > EntryEvent.getNewValue. Previously we were checking the > RegionEntry.isTombstone, but in the case of a PROXY region, the > RegionEntry is marker that does not set isTombstone to true. > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java > 46ccd47e96e75a8c4c6f3075b65a2d2447043213 > > geode-core/src/test/java/com/gemstone/gemfire/cache30/ClientServerCCEDUnitTest.java > 957dcc05d9d6481b95784a03ee6f2c52d5b937e5 > > Diff: https://reviews.apache.org/r/51510/diff/ > > > Testing > ------- > > > Thanks, > > Dan Smith > >