----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42465/#review115180 -----------------------------------------------------------
gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/versions/RegionVersionHolder.java (line 331) <https://reviews.apache.org/r/42465/#comment176077> Looks like in the recordVersionWithoutBitSet, in case when version == this.version and version= this.version + 1, we are still calling this.addOlderVersion(version). Would this have any unexpected impact on the end result? - Jinmei Liao On Jan. 18, 2016, 6:29 p.m., Bruce Schuchardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42465/ > ----------------------------------------------------------- > > (Updated Jan. 18, 2016, 6:29 p.m.) > > > Review request for geode, Dan Smith and xiaojian zhou. > > > Repository: geode > > > Description > ------- > > This is from a group refactoring session during a class taught by Michael > Feathers. The old recordVersion code was difficult to understand and was > poorly tested. New JUnit tests are now included that give 100% code coverage > for this algorithm. > > > Diffs > ----- > > > gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/versions/RegionVersionHolder.java > 4872df2a06c0bf6c949204cf98074d9259d6413f > > gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/versions/RegionVersionHolder2JUnitTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/42465/diff/ > > > Testing > ------- > > precheckin including new tests > > > Thanks, > > Bruce Schuchardt > >
