You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by cp...@apache.org on 2017/06/02 10:56:42 UTC
[19/38] lucene-solr:jira/solr-8668: SOLR-10137: Ensure that
ConfigSets created via API are mutable.
SOLR-10137: Ensure that ConfigSets created via API are mutable.
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/7da9e9d7
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/7da9e9d7
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/7da9e9d7
Branch: refs/heads/jira/solr-8668
Commit: 7da9e9d7352083060d45c1a10430a3a17fdeefbe
Parents: f34155f
Author: Mark Miller <ma...@apache.org>
Authored: Wed May 31 17:44:00 2017 -0400
Committer: Mark Miller <ma...@apache.org>
Committed: Wed May 31 17:44:08 2017 -0400
----------------------------------------------------------------------
solr/CHANGES.txt | 2 ++
.../java/org/apache/solr/handler/admin/ConfigSetsHandler.java | 4 ++++
.../src/test/org/apache/solr/handler/admin/TestConfigsApi.java | 2 +-
3 files changed, 7 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/7da9e9d7/solr/CHANGES.txt
----------------------------------------------------------------------
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index a2dc820..98f3ec2 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -252,6 +252,8 @@ Bug Fixes
* SOLR-10360: Solr HDFS snapshot export fails due to FileNotFoundException error when using MR1 instead of
yarn. (Hrishikesh via Mark Miller)
+
+* SOLR-10137: Ensure that ConfigSets created via API are mutable. (Hrishikesh via Mark Miller)
Optimizations
----------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/7da9e9d7/solr/core/src/java/org/apache/solr/handler/admin/ConfigSetsHandler.java
----------------------------------------------------------------------
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 3f857e7..34313d0 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 @@ public class ConfigSetsHandler extends RequestHandlerBase implements PermissionN
props.put(param, params.get(param));
}
}
+
+ // The configset created via an API should be mutable.
+ props.put("immutable", "false");
+
return props;
}
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/7da9e9d7/solr/core/src/test/org/apache/solr/handler/admin/TestConfigsApi.java
----------------------------------------------------------------------
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 d2c96a6..c0c193d 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 @@ public class TestConfigsApi extends SolrTestCaseJ4 {
"{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 }");
}
}