----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50967/#review145579 -----------------------------------------------------------
Ship it! Ship It! - Darrel Schneider On Aug. 11, 2016, 12:04 p.m., Scott Jewell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50967/ > ----------------------------------------------------------- > > (Updated Aug. 11, 2016, 12:04 p.m.) > > > Review request for geode, Darrel Schneider, Eric Shu, and Ken Howe. > > > Repository: geode > > > Description > ------- > > **I'LL SQUASH THIS WHEN I PUSH IT UP FOR THE PULL REQUEST** > > Cache modification lock was being released before operation distribution to > other members. > This provided a small window in which an operation from another thread could > update the > region prior to the other members receiving notification (i.e. down leveled) > > GEODE-599: Flush current operations if region scope is NO_ACK > > GEODE-599: Cleanup commented code > > GEODE-599: General code cleanup of ClearRvvLockingDUnitTest > > GEODE-599: Remove unnecessary region reference from RegionVersionVector > > commit > > > Merge remote-tracking branch 'origin/develop' into feature/GEODE-599 > > > Merge remote-tracking branch 'origin/develop' into feature/GEODE-599 > > > GEODE-599: Resolve issues with region operations and RVV locking > > > Merge remote-tracking branch 'origin/develop' into feature/GEODE-599 > > > GEODE-599: Add license to ARMLockTestHookAdapter > > > GEODE-599: Remove reference to test class VM from AbstractRegionMap > > > GEODE-599: Remove test class VM from ARM > > > GEODE-599: Refactor ARMLockTestHook interface > > > GEODE-599: Ensure test hook reset after test via finally clause > > > Merge remote-tracking branch 'origin/develop' into feature/GEODE-599 > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/AbstractRegionMap.java > 226e1948412a178c2a18a1e0a7d701b331c57e72 > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/BucketRegion.java > f1627d38b9d84c673a15c812712ebad297ce87ce > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/DistributedClearOperation.java > 5526ef01639869b10b0f8e475a6c23f62bdf9a86 > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/DistributedRegion.java > b42a617fe681700ca0c58f76c88b52535b7692b2 > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/LocalRegion.java > 46ccd47e96e75a8c4c6f3075b65a2d2447043213 > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/ProxyRegionMap.java > 3ad2cc1ff4fb352571bf6073d535fefb5837b12f > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/RegionMap.java > 14a2d2f09e2ff83b84080b2df558290a7a47710f > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/StateFlushOperation.java > 88666898574dfde3a2f7a148144f41f18f56687e > > geode-core/src/main/java/com/gemstone/gemfire/internal/cache/versions/RegionVersionVector.java > 2f024229d72c6259a360b1742fe33dbd8d6a2ab3 > > geode-core/src/test/java/com/gemstone/gemfire/internal/cache/ARMLockTestHookAdapter.java > PRE-CREATION > > geode-core/src/test/java/com/gemstone/gemfire/internal/cache/ClearRvvLockingDUnitTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/50967/diff/ > > > Testing > ------- > > > Thanks, > > Scott Jewell > >
