----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58010/ -----------------------------------------------------------
(Updated March 30, 2017, 7:19 p.m.) Review request for geode, Jinmei Liao, Jared Stewart, Kevin Duling, Ken Howe, Kirk Lund, and Swapnil Bawaskar. Changes ------- Added simpler class unit tests, updated variable name and location. Repository: geode Description (updated) ------- Added relevant unit and dUnit tests. Moved default value variable to ExportLogsCommand and deleted the now-unused LogService.DEFAULT_LOG_LEVEL Diffs (updated) ----- geode-core/src/main/java/org/apache/geode/internal/logging/LogService.java 1f8a564 geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ExportLogCommand.java 3f147c1 geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ExportLogsFunction.java cbdf1c4 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsDUnitTest.java 95edd42 geode-core/src/test/java/org/apache/geode/management/internal/cli/functions/ExportLogsFunctionTest.java PRE-CREATION Diff: https://reviews.apache.org/r/58010/diff/2/ Changes: https://reviews.apache.org/r/58010/diff/1-2/ Testing (updated) ------- precheckin running. Thanks, Patrick Rhomberg