----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55288/ -----------------------------------------------------------
Review request for geode, Jared Stewart, Kevin Duling, and Kirk Lund. Repository: geode Description ------- GEODE-2261: do not use remote function calls to update the shared configuration Diffs ----- geode-core/src/main/java/org/apache/geode/distributed/internal/DM.java 19381d4c951661b8581befc462044602c090cde2 geode-core/src/main/java/org/apache/geode/distributed/internal/DistributionManager.java 5ae54313eb16cf8613ca7dd884beb12c150a2b3b geode-core/src/main/java/org/apache/geode/distributed/internal/SharedConfiguration.java ec22b021675cd40d2a8383d13ede40fa757a2c6f geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/AbstractCommandsSupport.java 6db641569449c6cf789ccd8013ae524b7c34bcff geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ConfigCommands.java 4eac4e9ea0e5660d2f9ec39af30c5ba7e6ab8ff5 geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/CreateAlterDestroyRegionCommands.java 358cdc1795b5597995f7bfafd7a589127476d5f5 geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DeployCommands.java 7c887bee0a3346814b6961b4259ddbe668c1d4bd geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommands.java 505b6a830d986f367d267e543f2aa876e22c6470 geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ExportImportSharedConfigurationCommands.java 3a2d94975d4abf94348c9a18386956b4fe7c87a7 geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/IndexCommands.java adb03c6c301303e664c6e7bbdd658bb4cdd6bc84 geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/PDXCommands.java c5e9a4e26a790e20f14174fd3717db03f96dd697 geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/QueueCommands.java ebdae568492931643d6d4de68503985d3819854d geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/WanCommands.java dd6a327132a74cd2c481883c9c23ccb6c2d2f5a1 geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ExportSharedConfigurationFunction.java f20d9d444059e26b19f9468dbb07a0f10ab12e94 geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/ImportSharedConfigurationArtifactsFunction.java beee3db4e41d7748ee7ebeaa07b68ef3c7386b96 geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/LoadSharedConfigurationFunction.java c4fa6eeae319b52a90b90f3df37bbc00d9d2cfa9 geode-core/src/main/java/org/apache/geode/management/internal/cli/remote/RemoteExecutionStrategy.java 4ae1a1ca739ada17a33d554da856edf283e78b03 geode-core/src/main/java/org/apache/geode/management/internal/configuration/SharedConfigurationWriter.java 2be9873abc0d66506a6b63d58f9c5ff1f792b12f geode-core/src/main/java/org/apache/geode/management/internal/configuration/functions/AddJarFunction.java b986372f462cbfa30f7fe615e5b219ccff9db5e9 geode-core/src/main/java/org/apache/geode/management/internal/configuration/functions/AddXmlEntityFunction.java c562c9c1a9ea4213168560e30887d4f920d38f5a geode-core/src/main/java/org/apache/geode/management/internal/configuration/functions/DeleteJarFunction.java d0d36a86b810206d527b62cdcccc6840c616fb26 geode-core/src/main/java/org/apache/geode/management/internal/configuration/functions/DeleteXmlEntityFunction.java 1138d5ff8b277304882bc867aa160cd718ff8bb2 geode-core/src/main/java/org/apache/geode/management/internal/configuration/functions/ModifyXmlAndPropertiesFunction.java 97b9ec9fc4065eeeeb8141ed63df08566fd00658 geode-core/src/main/java/org/apache/geode/management/internal/configuration/functions/UploadJarFunction.java 7ea868264373cd83519f3bf1f9f21397569f3061 geode-core/src/main/java/org/apache/geode/management/internal/configuration/utils/XmlUtils.java cad770c50efa442dc6cb95734819a77e9570cfeb geode-core/src/test/java/org/apache/geode/distributed/internal/SharedConfigurationJUnitTest.java e3e5620610308a30552c14ee41ec607a979e6dcb geode-core/src/test/java/org/apache/geode/internal/cache/extension/mock/MockExtensionCommands.java e2a56fd8ce181b39d35f69db9ca191c28e43f7fe geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ConfigCommandsDUnitTest.java d86cb0be63caabb01179ed11b4015a3c3f172f32 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/CreateAlterDestroyRegionCommandsDUnitTest.java 7a75cfa7b402b921a2266a04ced9215428fe54b8 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DeployCommandsDUnitTest.java 7cf9a3a3fb6f58715919d8469365af6604352ef0 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommandsDUnitTest.java 901ed294c8867976c8cec272628b3943cb035f94 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/IndexCommandsDUnitTest.java 6dbd10c30342049c86980203dd40a28ae72fb9c3 geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/QueueCommandsDUnitTest.java b1897ba8da8e891eee85e53fb0a8dc1f2b9266e8 geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigBaseTest.java 28890b8a5bdb6acb9b8820d4c1d1964012e3ac10 geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigDeployJarDUnitTest.java a501235c8903889ab5506843c40df9145d51af48 geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigDistributionDUnitTest.java PRE-CREATION geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigImportDUnitTest.java cd8500af3d61cc8b5241dc1b7be8d3b142ef8b7a geode-core/src/test/java/org/apache/geode/management/internal/configuration/SharedConfigurationDUnitTest.java 2e733a7b188ed424717b92e15c7288d6902c5181 geode-core/src/test/java/org/apache/geode/management/internal/configuration/utils/XmlUtilsJUnitTest.java 3992146efe704c98e061739f4305647d2e1ded55 geode-core/src/test/java/org/apache/geode/redis/SetsJUnitTest.java 2d4a9291491933e893c3dd9fbc08aed12151ae21 geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java 9e46ad0cb284bcb50c9da3fc713b8617d10faeb5 geode-core/src/test/java/org/apache/geode/test/dunit/rules/LocatorServerConfigurationRule.java PRE-CREATION geode-core/src/test/java/org/apache/geode/test/dunit/rules/LocatorServerStartupRule.java 04fad5d974df3ded735ed24fe3509e0a06138416 geode-core/src/test/resources/org/apache/geode/distributed/internal/SharedConfigurationJUnitTest.xml geode-wan/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigurationDUnitTest.java f73a393cb4e293d3d743cd69eeaceafa4255959c geode-wan/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigurationExtensionsDUnitTest.java PRE-CREATION geode-wan/src/test/java/org/apache/geode/management/internal/configuration/WanDUnitTest.java PRE-CREATION Diff: https://reviews.apache.org/r/55288/diff/ Testing ------- precheckin running Thanks, Jinmei Liao