Null isn't supported

apache · Jan 30, 2025 · 010ae6f · 010ae6f
1 parent e428006
commit 010ae6f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/solr/core/src/java/org/apache/solr/handler/SchemaHandler.java b/solr/core/src/java/org/apache/solr/handler/SchemaHandler.java
@@ -16,6 +16,7 @@
  */
 package org.apache.solr.handler;
 
+import static java.util.Collections.singletonMap;
 import static org.apache.solr.common.params.CommonParams.JSON;
 import static org.apache.solr.schema.IndexSchema.SchemaProps.Handler.COPY_FIELDS;
 import static org.apache.solr.schema.IndexSchema.SchemaProps.Handler.DYNAMIC_FIELDS;
@@ -36,6 +37,7 @@
 import org.apache.solr.api.ApiBag;
 import org.apache.solr.api.JerseyResource;
 import org.apache.solr.common.SolrException;
+import org.apache.solr.common.params.MapSolrParams;
 import org.apache.solr.common.params.SolrParams;
 import org.apache.solr.common.util.StrUtils;
 import org.apache.solr.core.SolrCore;
@@ -168,7 +170,7 @@ private void handleGET(SolrQueryRequest req, SolrQueryResponse rsp) {
               if (parts.size() > 2) {
                 req.setParams(
                     SolrParams.wrapDefaults(
-                        SolrParams.of(pathParam, parts.get(2)), req.getParams()));
+                        new MapSolrParams(singletonMap(pathParam, parts.get(2))), req.getParams()));
               }
               switch (realName) {
                 case "fields":

diff --git a/solr/solrj/src/java/org/apache/solr/common/params/SolrParams.java b/solr/solrj/src/java/org/apache/solr/common/params/SolrParams.java
@@ -532,7 +532,7 @@ public static SolrParams of() {
     return EmptySolrParams.INSTANCE;
   }
 
-  /** An immutable SolrParams holding one pair. */
+  /** An immutable SolrParams holding one pair (not null). */
   public static SolrParams of(String k, String v) {
     return new MapSolrParams(Map.of(k, v));
   }