----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51380/#review146802 -----------------------------------------------------------
geode-core/src/main/java/com/gemstone/gemfire/internal/GemFireVersion.java (line 77) <https://reviews.apache.org/r/51380/#comment213486> This looks to be getting the wrong property, a duplicate 'SOURCE_REPOSITORY. Should this be changed to 'SOURCE_REVISION'? - Mark Bretl On Aug. 24, 2016, 10:36 a.m., Anthony Baker wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51380/ > ----------------------------------------------------------- > > (Updated Aug. 24, 2016, 10:36 a.m.) > > > Review request for geode, Dick Cavender and Jens Deppe. > > > Repository: geode > > > Description > ------- > > Clean up dead / broken / unused code and reduce the visible > API methods. Store the version info as a Map to allow it to be > extended more easily. > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/StartupMessage.java > 82f5c88c864fa7a03a7d33ee7cbcb1a08334667c > geode-core/src/main/java/com/gemstone/gemfire/internal/GemFireVersion.java > baf03b1d54bf0ac82024af63eb00d42331f6e8fb > geode-core/src/main/java/com/gemstone/gemfire/internal/SystemAdmin.java > e52950f69a48d0cb83dedb4e9ad42c287a1c49a9 > > geode-core/src/test/java/com/gemstone/gemfire/internal/GemFireVersionJUnitTest.java > fc4ab1b3c1c46bcb5c8f0e66f64c926fc37db789 > > geode-core/src/test/java/com/gemstone/gemfire/internal/VersionJUnitTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/51380/diff/ > > > Testing > ------- > > > Thanks, > > Anthony Baker > >
