----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56243/ -----------------------------------------------------------
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