----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57514/ -----------------------------------------------------------
Review request for geode, Jared Stewart, Kevin Duling, Ken Howe, Kirk Lund, and Patrick Rhomberg. Repository: geode Description ------- * correctly output error message if gfsh execution has an error * export logs should output correct log message over http connection as well Diffs ----- geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ExportLogCommand.java 36d071c153ed8c2559cc8c29e248058b42cbf7ff geode-core/src/main/java/org/apache/geode/management/internal/cli/shell/GfshExecutionStrategy.java ad78efd34fc4c9b1598751f71ba58ff5d29476a4 geode-core/src/main/java/org/apache/geode/management/internal/web/controllers/ExportLogController.java 0351573f6eda80e1751bd6b95efdbe6ce36a620d geode-core/src/main/java/org/apache/geode/management/internal/web/shell/AbstractHttpOperationInvoker.java f60aabceeae5d973caa900e5443b7cec0c0a75ca geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportStatsDUnitTest.java b7f8c2a080614ab59dc9da29c31f606749b58e4a Diff: https://reviews.apache.org/r/57514/diff/1/ Testing ------- precheckin running Thanks, Jinmei Liao