----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57573/#review168911 -----------------------------------------------------------
It looks to me like `DIR_OF_DELETED_FILES_IS_ONE_FILE` is unused. Did you intend to leave that around? - Jared Stewart On March 13, 2017, 11:31 p.m., Kirk Lund wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57573/ > ----------------------------------------------------------- > > (Updated March 13, 2017, 11:31 p.m.) > > > Review request for geode, Jinmei Liao, Jared Stewart, Kevin Duling, Ken Howe, > and Patrick Rhomberg. > > > Bugs: GEODE-2648 > https://issues.apache.org/jira/browse/GEODE-2648 > > > Repository: geode > > > Description > ------- > > GEODE-2648: always move deleted files to dirOfDeletedFiles > > Fix flaky test by always moving deleted files to dirOfDeletedFiles to avoid > race condition in which file is deleted before its presence can be detected. > > > Diffs > ----- > > > geode-core/src/test/java/org/apache/geode/internal/statistics/DiskSpaceLimitIntegrationTest.java > 5d94fa0ba32aabc0f0bfc396d99d0b5ef9572360 > > > Diff: https://reviews.apache.org/r/57573/diff/1/ > > > Testing > ------- > > DiskSpaceLimitIntegrationTest > precheckin in progress > > > Thanks, > > Kirk Lund > >
