pvcnt commented on code in PR #1973:
URL: https://github.com/apache/solr/pull/1973#discussion_r1360417880


##########
solr/core/src/java/org/apache/solr/cloud/overseer/ClusterStateMutator.java:
##########
@@ -125,6 +125,13 @@ public ZkWriteCommand createCollection(ClusterState 
clusterState, ZkNodeProps me
       collectionProps.put(ZkStateReader.CONFIGNAME_PROP, configName);
     }
 
+    // add user-defined properties
+    for (String prop : message.keySet()) {

Review Comment:
   I did not find those methods on a ZkNodeProps object, or did I overlooked 
something?



##########
solr/solr-ref-guide/modules/deployment-guide/pages/collection-management.adoc:
##########
@@ -250,6 +250,7 @@ If `true` the states of individual replicas will be 
maintained as individual chi
 +
 Set core property _name_ to _value_.
 See the section xref:configuration-guide:core-discovery.adoc[] for details on 
supported properties and values.

Review Comment:
   To confirm I understood well, are you mentioning the fact that we can add 
arbitrary properties (e.g., "foo" or "bar") this way?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to