SOLR-8615: Just like creating cores, we should use multiple threads when closing cores.

solr/CHANGES.txt

@@ -483,6 +483,9 @@ Optimizations
 
 * SOLR-8501: Specify the entity request size when known in HttpSolrClient. (Mark Miller)
 
+* SOLR-8615: Just like creating cores, we should use multiple threads when closing cores.
+  (Mark Miller)
+
 Other Changes
 ----------------------
 

solr/core/src/java/org/apache/solr/core/SolrCores.java

@@ -18,7 +18,9 @@ package org.apache.solr.core;
 
 import com.google.common.collect.Lists;
 import org.apache.solr.common.SolrException;
+import org.apache.solr.common.util.ExecutorUtil;
 import org.apache.solr.logging.MDCLoggingContext;
+import org.apache.solr.util.DefaultSolrThreadFactory;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -32,7 +34,10 @@ import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.TreeSet;
+import java.util.concurrent.Callable;
 import java.util.concurrent.ConcurrentHashMap;
+import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Future;
 import java.util.concurrent.TimeUnit;
 
 
@@ -119,17 +124,30 @@ class SolrCores {
       }
       
       for (SolrCore core : coreList) {
-        MDCLoggingContext.setCore(core);
+        ExecutorService coreCloseExecutor = ExecutorUtil.newMDCAwareFixedThreadPool(Integer.MAX_VALUE,
+            new DefaultSolrThreadFactory("coreCloseExecutor"));
         try {
-          core.close();
+          coreCloseExecutor.submit(new Callable<SolrCore>() {
-        } catch (Throwable e) {
+            @Override
-          SolrException.log(log, "Error shutting down core", e);
+            public SolrCore call() throws Exception {
-          if (e instanceof Error) {
+              MDCLoggingContext.setCore(core);
-            throw (Error) e;
+              try {
-          }
+                core.close();
+              } catch (Throwable e) {
+                SolrException.log(log, "Error shutting down core", e);
+                if (e instanceof Error) {
+                  throw (Error) e;
+                }
+              } finally {
+                MDCLoggingContext.clear();
+              }
+              return core;
+            }
+          });
         } finally {
-          MDCLoggingContext.clear();
+          ExecutorUtil.shutdownAndAwaitTermination(coreCloseExecutor);
         }
+
       }
     } while (coreList.size() > 0);
   }