----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53857/ -----------------------------------------------------------
Review request for geode. Repository: geode Description ------- Changed checks for expected log messages from time based to event based. Tests that verify that log messages occured as expected were using an overly optimistic test for 'n' logs after a set period of time. The Mockito.verify calls are now placed within the swait()'s. Also simplified tests that have a final check to make sure missing colocated region logging stopped after all child PRs are created. Diffs ----- geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentColocatedPartitionedRegionDUnitTest.java a044a612fbbc6b905e3ad1389a9c0d7994f8598a Diff: https://reviews.apache.org/r/53857/diff/ Testing ------- Ran spotlessCheck (and spotlessApply to fix some formatting) Precheckin is in progress Thanks, Ken Howe
