----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50967/ -----------------------------------------------------------
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
