----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59811/ -----------------------------------------------------------
(Updated June 7, 2017, 6:35 p.m.) Review request for geode, Emily Yeh, Jinmei Liao, Jared Stewart, Kirk Lund, and Patrick Rhomberg. Changes ------- Revised the semantics of the export logs size warning: Previous: - file-size-limit option was for the size of the zip file that is exported to the locator. To make this check, all the work of exporting the logs was completed even thought the exported zipfile is eventually deleted if it was too big. - file-size-limit > 0 also enabled checking for potential disk overflow on each member as well as on the locator. This check use the estimated disk space required to filter and consolidate the fully expanded individual log files. - file-size-limit = 0 disables all checking of exported logs size and potential disk overflows Revised: - file-size-limit option will check that the estimated size of all of the fully expanded log files as exported to the locator is less than the specified limit (or the default of 100m if not specified). The work (and temporary disk space) of preparing and exporting the logs from all the members is not performed if the check fails (expnaded size on the locator is too big). - file-size-limit = 0 - no changes Refactored SizeExportLogsFunction. - Simplified the Result returned by the function. The result will always be a single Long (the estimated expnaded size of the filtered logs for the member) or an error result. Fixed the issue with file-size-limit not being recognized when connected to the locator via http Repository: geode Description ------- GEODE-2420: add file-size-limit param to the ExportLogsController Diffs (updated) ----- geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ExportLogsCommand.java 0ff780cbf66937d8ececfb3a2d0789ee485b9b62 geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/SizeExportLogsFunction.java 57355c0efae4c6da9470267f95e27e59aa4d8b2c geode-core/src/main/java/org/apache/geode/management/internal/cli/i18n/CliStrings.java 9f68d3a5eaadbe8f1bd95ec8df85ed1f65aa67ce geode-core/src/main/java/org/apache/geode/management/internal/web/controllers/ExportLogController.java a369c6e1ffb330715fbde2cd69d023ed36f133ad geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsCommandTest.java 16549e70bbebf4390bb73a481274e92ca6cad035 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsDUnitTest.java 8609b3aaf0a0eb1ba903bd39c64103f9510a6a78 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsFileSizeLimitTest.java 09ee08dd6af29b9a418ef7499defc4980da787ed geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsStatsDUnitTest.java 44a036298e0991c880fc552596d296e104b97ca1 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsTestSuite.java 4e1dac013d239437829bc52dc70689c4ba15dc58 geode-core/src/test/java/org/apache/geode/management/internal/cli/functions/SizeExportLogsFunctionTest.java cc5e7d5256741ad0a48ff87c7f989a18b90f7f03 Diff: https://reviews.apache.org/r/59811/diff/2/ Changes: https://reviews.apache.org/r/59811/diff/1-2/ Testing (updated) ------- 6/7/17: re-started precheckin Precheckin started Thanks, Ken Howe