----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61895/#review183945 -----------------------------------------------------------
geode-core/src/test/java/org/apache/geode/internal/cache/TXManagerImplJUnitTest.java Lines 380 (patched) <https://reviews.apache.org/r/61895/#comment259991> What does this Thread.sleep() do, if the next line is awaiting a latch? geode-core/src/test/java/org/apache/geode/internal/cache/TXManagerImplJUnitTest.java Lines 384 (patched) <https://reviews.apache.org/r/61895/#comment259995> Is spyMgr.suspend() here being attempted to occur between when the two threads start and before they complete? If so, is that guaranteed to occur? - Nick Reich On Aug. 25, 2017, 4:53 p.m., Eric Shu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61895/ > ----------------------------------------------------------- > > (Updated Aug. 25, 2017, 4:53 p.m.) > > > Review request for geode, anilkumar gingade, Darrel Schneider, Lynn Gallinat, > and Nick Reich. > > > Bugs: GEODE-3516 > https://issues.apache.org/jira/browse/GEODE-3516 > > > Repository: geode > > > Description > ------- > > Remove the thread from waiting thread queue after successfully resumed the > transaction > > > Diffs > ----- > > geode-core/src/main/java/org/apache/geode/internal/cache/TXManagerImpl.java > a0a4d7c > > geode-core/src/test/java/org/apache/geode/internal/cache/TXManagerImplJUnitTest.java > a2c1e70 > > > Diff: https://reviews.apache.org/r/61895/diff/2/ > > > Testing > ------- > > precheckin. > > > Thanks, > > Eric Shu > >
