----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58162/#review171074 -----------------------------------------------------------
Ship it! Ship It! - Kirk Lund On April 4, 2017, 10:02 p.m., Jinmei Liao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58162/ > ----------------------------------------------------------- > > (Updated April 4, 2017, 10:02 p.m.) > > > Review request for geode, Jared Stewart, Ken Howe, Kirk Lund, and Patrick > Rhomberg. > > > Repository: geode > > > Description > ------- > > * move ImportClusterConfigDistributedTest to open side > * remove @Ignore from other two tests. > * force the isGfshVM flag to be false before exeuting the command since class > loading issue still contaminate the close > side test environment. > > > Diffs > ----- > > > geode-core/src/test/java/org/apache/geode/management/internal/configuration/ImportClusterConfigDistributedTest.java > PRE-CREATION > > geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java > 6ae82da58923fdf3a44217c1773a40d8ff1001bc > > geode-core/src/test/java/org/apache/geode/test/dunit/rules/LocatorServerStartupRule.java > 2d648bd599c57b91252aaf37bce4bf0635c4eda6 > > > Diff: https://reviews.apache.org/r/58162/diff/2/ > > > Testing > ------- > > precheckin running > > > Thanks, > > Jinmei Liao > >
