----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60710/#review179944 -----------------------------------------------------------
geode-core/src/test/java/org/apache/geode/internal/cache/ClientServerTransactionDUnitTest.java Line 3991 (original), 3992 (patched) <https://reviews.apache.org/r/60710/#comment254883> Can you add a comment why it should wait for states to be empty? - Darrel Schneider On July 7, 2017, 9:51 a.m., Eric Shu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60710/ > ----------------------------------------------------------- > > (Updated July 7, 2017, 9:51 a.m.) > > > Review request for geode, anilkumar gingade, Darrel Schneider, and Lynn > Gallinat. > > > Bugs: GEODE-3015 > https://issues.apache.org/jira/browse/GEODE-3015 > > > Repository: geode > > > Description > ------- > > Fix a flaky unit test > > > Diffs > ----- > > > geode-core/src/test/java/org/apache/geode/internal/cache/ClientServerTransactionDUnitTest.java > ec4e179 > > > Diff: https://reviews.apache.org/r/60710/diff/1/ > > > Testing > ------- > > Unit test. > > > Thanks, > > Eric Shu > >