> On Feb. 8, 2017, 7:44 p.m., Dan Smith wrote: > > geode-core/src/test/java/org/apache/geode/test/dunit/standalone/VersionManager.java, > > line 155 > > <https://reviews.apache.org/r/56457/diff/1/?file=1627828#file1627828line155> > > > > Is this method still needed?
loadProperties() isn't needed anymore. I'll delete it. - Bruce ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56457/#review164748 ----------------------------------------------------------- On Feb. 8, 2017, 7:01 p.m., Bruce Schuchardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56457/ > ----------------------------------------------------------- > > (Updated Feb. 8, 2017, 7:01 p.m.) > > > Review request for geode and Dan Smith. > > > Bugs: GEODE-2433 > https://issues.apache.org/jira/browse/GEODE-2433 > > > Repository: geode > > > Description > ------- > > Modified to look for the file on the class-path instead of poking around in > the file system > > > Diffs > ----- > > > geode-core/src/test/java/org/apache/geode/test/dunit/standalone/VersionManager.java > c06d4e8fadf7aaba5432280195ef8524e6aadcb5 > > Diff: https://reviews.apache.org/r/56457/diff/ > > > Testing > ------- > > Tested under Eclipse, Idea and Gradle > > > Thanks, > > Bruce Schuchardt > >