murblanc commented on a change in pull request #2179:
URL: https://github.com/apache/lucene-solr/pull/2179#discussion_r553633390
##########
File path:
solr/core/src/java/org/apache/solr/cloud/api/collections/CreateCollectionCmd.java
##########
@@ -116,6 +121,25 @@ public void call(ClusterState clusterState, ZkNodeProps
message, @SuppressWarnin
throw new SolrException(SolrException.ErrorCode.BAD_REQUEST, "collection
alias already exists: " + collectionName);
}
+ String withCollection = message.getStr(WITH_COLLECTION);
+ String withCollectionShard = null;
+ if (withCollection != null) {
+ String realWithCollection = aliases.resolveSimpleAlias(withCollection);
Review comment:
We should not follow aliases at all and specify that the
`withCollection` must be an existing real collection name.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
