----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57522/ -----------------------------------------------------------
(Updated March 13, 2017, 10:31 p.m.) Review request for geode, Jinmei Liao, Jared Stewart, Kevin Duling, Kirk Lund, Patrick Rhomberg, and Swapnil Bawaskar. Summary (updated) ----------------- check for string "null" in server/locator status message Repository: geode Description (updated) ------- Restored original functionality when launcher process exits. Unintentionally omitted this when refactoring from using exitValue() to isAlive() Check for string "null" in server/locator status message to prevent interrupting the string of dots. Refactored server and locator start commands to use JDK 1.8 Process.isAlive instead of a try/catch with Process.exitValue Diffs (updated) ----- geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/LauncherLifecycleCommands.java d42d75e Diff: https://reviews.apache.org/r/57522/diff/2/ Changes: https://reviews.apache.org/r/57522/diff/1-2/ Testing (updated) ------- Ran spotlessApply Re-started precheckin Thanks, Ken Howe
