----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55532/ -----------------------------------------------------------
(Updated Jan. 17, 2017, 5:07 a.m.) Review request for geode, Jared Stewart, Kevin Duling, and Kirk Lund. Repository: geode Description ------- * When importing cluster config first check if there is any non-empty region * close and re-create cache if no data exists when importing new cluster configuration * put the acquire/release lock inside the ClusterConfigurationService instead of command execution strategy. Diffs (updated) ----- geode-core/src/main/java/org/apache/geode/distributed/internal/ClusterConfigurationService.java 1d4030a8dedd017a0ab096925055f375bb5d3ef0 geode-core/src/main/java/org/apache/geode/internal/cache/GemFireCacheImpl.java 795164d7a86f87afb60e800885fb5fb2c540451b geode-core/src/main/java/org/apache/geode/management/cli/CliMetaData.java 2e272fc083514456f0ef89d4b9824126e86828fa geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/CreateAlterDestroyRegionCommands.java 7df4112950cefcc1ae18bdd88cb752cdec5b7de1 geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DeployCommands.java f076cec670a4c787fbe4002edfdacde4b025d1bf geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommands.java 14114cff2febf0e1ba7ae67427549ad4fd19ed85 geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ExportImportSharedConfigurationCommands.java 914576b2b24eec670c06510aa5b056c0acd808fc geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/IndexCommands.java bc436ba40e107b931e8d0d10082e26cf068dd56b geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/PDXCommands.java 72d5c0fc65090880b95f37868e72ef9dcba6c55f geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/QueueCommands.java 095bd68f54d0f06d9f9439fcaffbca72e49083fc geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/WanCommands.java cbf589185e94345be6fd64ebd630f76e84cc2a09 geode-core/src/main/java/org/apache/geode/management/internal/cli/remote/RemoteExecutionStrategy.java 1e4870f9b48e5cc5e4c7ded2af907f4e63aa2977 geode-core/src/main/java/org/apache/geode/management/internal/configuration/functions/RecreateCacheFunction.java PRE-CREATION geode-core/src/main/java/org/apache/geode/management/internal/configuration/functions/RegionsWithDataOnServerFunction.java PRE-CREATION geode-core/src/test/java/org/apache/geode/internal/cache/extension/mock/MockExtensionCommands.java 1bd4478538f0e0c9413e95a01e835fa6af3a24ea geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ClusterConfigurationServiceCommandsDUnitTest.java 87cac55bc672ed380d3abb8ab69c9499f931e4b3 geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfig.java c2d511d1c4d34aea434751c2bb5b7e9b924482ad geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigBaseTest.java 506428a48c27435a3c89f97bc4d296853fe99415 geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigImportDUnitTest.java b301b80a1268c8ad0954e9abe86711cdfe5ac066 geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigStartMemberDUnitTest.java 7dd0da67bdb38f027f717bfc96adc5f5ffc0af37 geode-core/src/test/java/org/apache/geode/test/dunit/rules/ServerStarterRule.java c94185a3b7f8e82cdab620b79c3cb836d0a7e530 geode-core/src/test/resources/org/apache/geode/codeAnalysis/sanctionedSerializables.txt 9626be76c35194e07df10d3ad650cb3d2df4d73a geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal/cli/LuceneIndexCommands.java 6a5a1e09c03b9a5362e44fd6fe55a6e6181ea17c geode-web/src/test/java/org/apache/geode/management/internal/cli/commands/CommandOverHttpDUnitTest.java a1810b6eaa7d940ad359d64faab98c7d113165f0 Diff: https://reviews.apache.org/r/55532/diff/ Testing ------- precheckin successful Thanks, Jinmei Liao
