----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60974/ -----------------------------------------------------------
(Updated July 20, 2017, 9:51 p.m.) Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and Patrick Rhomberg. Repository: geode Description (updated) ------- * do not log to file by default, only when withLogInFile is called * do not create a workingDir and sets the user.dir by default, only when withWorkingDir is called * For LocatorServerStarterRule, also do not set the workingDir if not asked to, and only bounce the VM if the VM is started with a workingDir Diffs (updated) ----- geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsDUnitTest.java 045e13edc798f2bd3460dc17bbfeec28ac5022e4 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsIntegrationTest.java 4ca625a436ad9d181f1fecd11afc7a0cf1157aa6 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsOnServerManagerDUnit.java a78b26fb560c17c96b794ecedc09e10c87766ef0 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsStatsDUnitTest.java 33439e6063150b87938f21f13e647d3b0ecc4896 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportLogsWithMemberGroupDUnitTest.java e5cca279d8a8082726fc67eab8a5cfd91ce90529 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ShowMissingDiskStoresDUnitTest.java 766f25ada3b446df71d840d70bb6fae797a85a00 geode-core/src/test/java/org/apache/geode/management/internal/cli/functions/ExportLogsFunctionIntegrationTest.java d98641849eed458b4c830e1fa198f4d986b0c4ce geode-core/src/test/java/org/apache/geode/management/internal/cli/functions/SizeExportLogsFunctionTest.java 510a4e7fef1aff5a1aa8f00f569cd6422b7816ab geode-core/src/test/java/org/apache/geode/management/internal/cli/util/LogExporterIntegrationTest.java a8b498ddbecb9d8733ed4f98c1c80cb18da5c37e geode-core/src/test/java/org/apache/geode/management/internal/cli/util/MergeLogsDUnitTest.java 9166642044055071b2cced88edfe3ae660976ef7 geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigImportDUnitTest.java 066f8828f64b7caa9c9748683f9214868105fed8 geode-core/src/test/java/org/apache/geode/management/internal/security/MemberMBeanSecurityJUnitTest.java e6b50d210dad844aca5e37fcb3cf58d7aace4b3e geode-core/src/test/java/org/apache/geode/security/ClusterConfigWithoutSecurityDUnitTest.java e90bc0a69222998322e02fcfad1b6bad3c97f4d1 geode-core/src/test/java/org/apache/geode/security/PDXPostProcessorDUnitTest.java c4a25470bab75a2fdfcdb55126564d21ecd9403d geode-core/src/test/java/org/apache/geode/security/PeerAuthenticatorDUnitTest.java b977b50e28b8d518fd3d9da6c03b55f8b7cbff34 geode-core/src/test/java/org/apache/geode/test/dunit/rules/LocatorServerStartupRule.java 62ae2e223f019c78000ff7492510c24aba9c930c geode-core/src/test/java/org/apache/geode/test/dunit/rules/MemberStarterRule.java 226fd6c24bb1ef818fd2de2bd010d713b59872e8 geode-core/src/test/java/org/apache/geode/test/dunit/rules/MemberVM.java 7e5ce1f1e41eff125c5d2e75f389f5ea8b1f90ed geode-core/src/test/java/org/apache/geode/test/dunit/rules/ServerStarterRule.java 6ea2d0336f74956ff81efee94fc1ac2504383520 geode-core/src/test/java/org/apache/geode/test/dunit/rules/test/MemberStarterRuleTest.java 7dada04cc63aaa2668e103524bd63d3c58310257 geode-web/src/test/java/org/apache/geode/management/internal/security/LogNoPasswordTest.java 3176f540e5605989a125d9e3b533759ce009ed42 Diff: https://reviews.apache.org/r/60974/diff/3/ Changes: https://reviews.apache.org/r/60974/diff/2-3/ Testing ------- Thanks, Jinmei Liao
