----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59542/ -----------------------------------------------------------
(Updated May 30, 2017, 5:54 p.m.) Review request for geode, Emily Yeh, Jinmei Liao, Jared Stewart, Kirk Lund, and Patrick Rhomberg. Changes ------- Reduces scope of the change to ResultBuilder class; the original method names have been restored. Error message text removes mention of "GemFire" or "Geode" to now report: "Could not process command due to error." plus the message passed to the ResultBuilder Repository: geode Description ------- Change GemFire to Geode in method names and message string This is change to simply correct the Gfsh command error strings where "GemFire" appears. With additional work on Gfsh command error handling pending (see GEODE-2984), no attempt has been made within the scope of this issue to reorganize or otherwise improve Gfsh command error messages. Diffs (updated) ----- geode-core/src/main/java/org/apache/geode/management/internal/cli/result/ResultBuilder.java 6332540aa4ff6d210ed7f3ba167057de9a0a5023 Diff: https://reviews.apache.org/r/59542/diff/2/ Changes: https://reviews.apache.org/r/59542/diff/1-2/ Testing (updated) ------- 5/30 - precheckin restarted precheckin is running Thanks, Ken Howe