----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61084/ -----------------------------------------------------------
Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and Patrick Rhomberg. Repository: geode Description ------- More refactor of the rules GEODE-3231: withLogFile does not imply withWorkingDir anymore Diffs ----- geode-core/src/test/java/org/apache/geode/internal/ClassPathLoaderIntegrationTest.java 721e5020ead8c93313008d11599195a0a0022f9e geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsDUnitTest.java 492e304e43113877102be22304bab09025a36dbc geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsIntegrationTest.java 6726ede5e7a2e9237406d701e8d01c8cbb4adf8b geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsOnServerManagerDUnit.java 557fae7f58d5b7eb29818177c329de255eea75c1 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsStatsDUnitTest.java a1dff25990fcf1fd1c41e2e291d4ef277032ba36 geode-core/src/test/java/org/apache/geode/management/internal/cli/functions/ExportLogsFunctionIntegrationTest.java 9669669c4fe103ddb21885eca5f23743f75ae005 geode-core/src/test/java/org/apache/geode/management/internal/cli/functions/ExportLogsFunctionTest.java 4e72444d634ab37d39b3b01018d5f21c6ce67753 geode-core/src/test/java/org/apache/geode/management/internal/cli/functions/SizeExportLogsFunctionTest.java 12410c79a3df1624e165ebecf1f97425e1ce08e6 geode-core/src/test/java/org/apache/geode/management/internal/cli/util/LogExporterIntegrationTest.java 0cd6d5af65aab3d286267b663418b798da362904 geode-core/src/test/java/org/apache/geode/management/internal/cli/util/MergeLogsDUnitTest.java 35d65a005198b981581bf7d02476ad79cd0eea9c geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java 872553efed290890708caaf98dc206b4735005d0 geode-core/src/test/java/org/apache/geode/test/dunit/rules/LocatorServerStartupRule.java d8753cf285388acabef91761a5043ee18f8475f3 geode-core/src/test/java/org/apache/geode/test/dunit/rules/LocatorStarterRule.java e1c14ce828d45d3777e9b31f4b7019e23d6a19ce geode-core/src/test/java/org/apache/geode/test/dunit/rules/MemberStarterRule.java 156236bae5cc1b5dc477b01e8b1955bf4aae64fb geode-core/src/test/java/org/apache/geode/test/dunit/rules/ServerStarterRule.java 0fd0e7759695d05071a4ade327b129107a9d9d0e geode-core/src/test/java/org/apache/geode/test/dunit/rules/test/MemberStarterRuleTest.java 4970cb5a6becfe259c70132adad949d34f7e2952 geode-web/src/test/java/org/apache/geode/management/internal/security/LogNoPasswordTest.java 368d64dd9c2be3fb15d6d4563bb1c95d83733244 Diff: https://reviews.apache.org/r/61084/diff/1/ Testing ------- precheckin running Thanks, Jinmei Liao