----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55106/#review160435 -----------------------------------------------------------
Ship it! Ship It! - Udo Kohlmeyer On Jan. 3, 2017, 6:03 p.m., Bruce Schuchardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55106/ > ----------------------------------------------------------- > > (Updated Jan. 3, 2017, 6:03 p.m.) > > > Review request for geode, Galen O'Sullivan, Hitesh Khamesra, and Udo > Kohlmeyer. > > > Bugs: GEODE-1580 > https://issues.apache.org/jira/browse/GEODE-1580 > > > Repository: geode > > > Description > ------- > > Added a dunit blackboard to coordinate creation of a deadlock with the > DistributedLockService. Removed pauses and added use of Awaitility to wait > for conditions. > > > Diffs > ----- > > > geode-core/src/test/java/org/apache/geode/distributed/internal/deadlock/GemFireDeadlockDetectorDUnitTest.java > 17ec800517066b1a7564a91be10c7d659f24a8fb > geode-core/src/test/java/org/apache/geode/test/dunit/DUnitBlackboard.java > PRE-CREATION > > geode-core/src/test/java/org/apache/geode/test/dunit/internal/DUnitBlackboardDUnitTest.java > PRE-CREATION > > geode-core/src/test/java/org/apache/geode/test/dunit/internal/InternalBlackboard.java > PRE-CREATION > > geode-core/src/test/java/org/apache/geode/test/dunit/internal/InternalBlackboardImpl.java > PRE-CREATION > > geode-core/src/test/java/org/apache/geode/test/dunit/internal/JUnit4DistributedTestCase.java > ebbb2fd5fffd5b4e1120a05c5103698b6be9e1f5 > > geode-core/src/test/java/org/apache/geode/test/dunit/standalone/ChildVM.java > 7a9d7f00ff2efcdb811217c5dd42baeebe862685 > > geode-core/src/test/java/org/apache/geode/test/dunit/standalone/DUnitLauncher.java > 5a80a64f82ad0d7ed6e22c06e006e82d7d1e9d95 > > Diff: https://reviews.apache.org/r/55106/diff/ > > > Testing > ------- > > Running precheckin since some dunit infrastructure was modified > > > Thanks, > > Bruce Schuchardt > >