From 2e37a0bb5600419591428b9d985f68bfd79cfe06 Mon Sep 17 00:00:00 2001 From: Hrishikesh Gadre Date: Thu, 25 May 2017 11:13:02 -0700 Subject: [PATCH] [SOLR-10137] Ensure that ConfigSet created via an API is mutable --- .../java/org/apache/solr/handler/admin/ConfigSetsHandler.java | 4 ++++ .../test/org/apache/solr/handler/admin/TestConfigsApi.java | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/solr/core/src/java/org/apache/solr/handler/admin/ConfigSetsHandler.java b/solr/core/src/java/org/apache/solr/handler/admin/ConfigSetsHandler.java index 3f857e79d67f..34313d00d892 100644 --- a/solr/core/src/java/org/apache/solr/handler/admin/ConfigSetsHandler.java +++ b/solr/core/src/java/org/apache/solr/handler/admin/ConfigSetsHandler.java @@ -238,6 +238,10 @@ private static Map copyPropertiesWithPrefix(SolrParams params, M props.put(param, params.get(param)); } } + + // The configset created via an API should be mutable. + props.put("immutable", "false"); + return props; } diff --git a/solr/core/src/test/org/apache/solr/handler/admin/TestConfigsApi.java b/solr/core/src/test/org/apache/solr/handler/admin/TestConfigsApi.java index d2c96a66be2a..c0c193de872b 100644 --- a/solr/core/src/test/org/apache/solr/handler/admin/TestConfigsApi.java +++ b/solr/core/src/test/org/apache/solr/handler/admin/TestConfigsApi.java @@ -54,6 +54,6 @@ protected void sendToZk(SolrQueryResponse rsp, "{name :sample, operation:delete}"); compareOutput(apiBag, "/cluster/configs", POST, "{create:{name : newconf, baseConfigSet: sample }}", null, - "{operation:create, name :newconf, baseConfigSet: sample }"); + "{operation:create, name :newconf, baseConfigSet: sample, immutable: false }"); } }