Cleanup

cowpaths · Feb 26, 2024 · c69deb9 · c69deb9
1 parent 959ab57
commit c69deb9
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 8 deletions.
diff --git a/solr/core/src/java/org/apache/solr/core/CoreContainer.java b/solr/core/src/java/org/apache/solr/core/CoreContainer.java
@@ -1640,7 +1640,7 @@ public AllowListUrlChecker getAllowListUrlChecker() {
    * @return the newly created core
    */
   @SuppressWarnings("resource")
-  public SolrCore createFromDescriptor(
+  private SolrCore createFromDescriptor(
       CoreDescriptor dcore, boolean publishState, boolean newCollection) {
 
     if (isShutDown) {

diff --git a/solr/core/src/java/org/apache/solr/core/SolrCoreProxy.java b/solr/core/src/java/org/apache/solr/core/SolrCoreProxy.java
@@ -11,20 +11,15 @@ public SolrCoreProxy(CoreContainer coreContainer, CoreDescriptor cd, ConfigSet c
   }
 
   public static SolrCoreProxy createAndRegisterProxy(
-      CoreContainer coreContainer,
-      String syntheticCollectionName,
-      String configSetName,
-      String todo) {
+      CoreContainer coreContainer, String syntheticCollectionName, String configSetName) {
     Map<String, String> coreProps = new HashMap<>();
     coreProps.put(CoreAdminParams.CORE_NODE_NAME, coreContainer.getHostName());
     coreProps.put(CoreAdminParams.COLLECTION, syntheticCollectionName);
 
     CoreDescriptor syntheticCoreDescriptor =
         new CoreDescriptor(
             syntheticCollectionName,
-            //                    todo,
             Paths.get(coreContainer.getSolrHome() + "/" + syntheticCollectionName),
-            //                    Paths.get(coreContainer.getSolrHome() + "/" + todo),
             coreProps,
             coreContainer.getContainerProperties(),
             coreContainer.getZkController());

diff --git a/solr/core/src/java/org/apache/solr/servlet/CoordinatorHttpSolrCall.java b/solr/core/src/java/org/apache/solr/servlet/CoordinatorHttpSolrCall.java
@@ -89,7 +89,7 @@ public static SolrCore getCore(
         CoreContainer coreContainer = solrCall.cores;
         SolrCoreProxy syntheticCore =
             SolrCoreProxy.createAndRegisterProxy(
-                coreContainer, syntheticCollectionName, coll.getConfigName(), collectionName);
+                coreContainer, syntheticCollectionName, coll.getConfigName());
 
         // after this point the sync core should be available in the container. Double check
         if (coreContainer.getCore(syntheticCore.getName()) != null) {