----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53154/#review153793 -----------------------------------------------------------
So what did you decide with regard to the await timeout? You noted a change to 60 but your commit left it at 30. - Ken Howe On Oct. 24, 2016, 11:19 p.m., Darrel Schneider wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53154/ > ----------------------------------------------------------- > > (Updated Oct. 24, 2016, 11:19 p.m.) > > > Review request for geode, anilkumar gingade, Eric Shu, Scott Jewell, and Ken > Howe. > > > Bugs: GEODE-2026 > https://issues.apache.org/jira/browse/GEODE-2026 > > > Repository: geode > > > Description > ------- > > The test was initializing a value to a String, > then starting an async thread that would keep > changing it to an int, > it slept for 100ms and then assummed that the async > thread had changed the value to an Integer. > In rare cases the thread never was able to to its first > put so the value was still a String. > > The test now waits for the async thread to change the value > to an Integer instead of doing a 100ms sleep. > > > Diffs > ----- > > > geode-core/src/test/java/org/apache/geode/internal/cache/partitioned/PersistentPartitionedRegionDUnitTest.java > 5ca3d014b4bac01e239561301666fae5fb0d61ea > > Diff: https://reviews.apache.org/r/53154/diff/ > > > Testing > ------- > > precheckin > > > Thanks, > > Darrel Schneider > >
