----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56243/#review164031 -----------------------------------------------------------
Ship it! Ship It! - Jared Stewart On Feb. 2, 2017, 9:38 p.m., Dan Smith wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/56243/ > ----------------------------------------------------------- > > (Updated Feb. 2, 2017, 9:38 p.m.) > > > Review request for geode and Kirk Lund. > > > Repository: geode > > > Description > ------- > > Fixing DistributedRestoreSystemProperties rule so that if the rule is > included twice within a test, it does not end up calling > System.setProperties(null). That will prevent us from losing the value > of java.class.path set by gradle. > > Reverting the workaround that didn't actually work. > > > Diffs > ----- > > > geode-core/src/test/java/org/apache/geode/test/dunit/rules/DistributedRestoreSystemProperties.java > 7e6198e865e33908c0e89f4e0a0c20328f56d55e > > geode-core/src/test/java/org/apache/geode/test/dunit/standalone/ProcessManager.java > 3b02b4b5e320849e431e9f6720451452639d4c65 > > Diff: https://reviews.apache.org/r/56243/diff/ > > > Testing > ------- > > > Thanks, > > Dan Smith > >
