----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61701/ -----------------------------------------------------------
(Updated Aug. 22, 2017, 9:28 p.m.) Review request for geode, Emily Yeh, Jinmei Liao, Jared Stewart, Kirk Lund, and Patrick Rhomberg. Changes ------- Refactored ServerLauncher similar to the previous change to LocatorLauncher to remove nested ternary expressions. Repository: geode Description ------- Updated tests for changes in the error constructors for ServerState and LocatorState. Minor spelling corrections. Diffs (updated) ----- geode-assembly/src/test/java/org/apache/geode/management/internal/cli/commands/StatusLocatorRealGfshTest.java 3a98373938e3de21da6badcf460dae3648218ac6 geode-core/src/main/java/org/apache/geode/distributed/LocatorLauncher.java 83c1ab533e3dea323a8a99f7002b9464a54dfc25 geode-core/src/main/java/org/apache/geode/distributed/ServerLauncher.java ae64691605130c9b212a3a33bb65ae37b28af02b geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/lifecycle/GfshStatusCommandsIntegrationTest.java dd5841f4cffca38da07a11f381cf4174d7264349 geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java e7f17ef208a1708f385c7c4041affb70fd309a4c Diff: https://reviews.apache.org/r/61701/diff/3/ Changes: https://reviews.apache.org/r/61701/diff/2-3/ Testing (updated) ------- Precheckin from earlier ran green. Re-running precheckin with this additional refactoring. Thanks, Ken Howe
