Address review around map

Author: pawankartik-elastic

server/src/main/java/org/elasticsearch/action/search/SearchResponse.java

@@ -615,7 +615,8 @@ public Clusters(Map<String, Cluster> clusterInfoMap) {
         public Clusters(Map<String, Cluster> clusterInfoMap, boolean ccsMinimizeRoundtrips) {
             assert clusterInfoMap.size() > 0 : "this constructor should not be called with an empty Cluster info map";
             this.total = clusterInfoMap.size();
-            this.clusterInfo = clusterInfoMap;
+            this.clusterInfo = ConcurrentCollections.newConcurrentMap();
+            this.clusterInfo.putAll(clusterInfoMap);
             this.successful = getClusterStateCount(Cluster.Status.SUCCESSFUL);
             this.skipped = getClusterStateCount(Cluster.Status.SKIPPED);
             // should only be called if "details" section of fromXContent is present (for ccsMinimizeRoundtrips)

server/src/main/java/org/elasticsearch/action/search/TransportSearchAction.java

@@ -1020,7 +1020,7 @@ static SearchResponse.Clusters reconcileProjects(
                 expr -> expr.localExpressions().localIndexResolutionResult() == ResolvedIndexExpression.LocalIndexResolutionResult.SUCCESS
             );
 
-        Map<String, SearchResponse.Cluster> reconciledMap = ConcurrentCollections.newConcurrentMap();
+        Map<String, SearchResponse.Cluster> reconciledMap = new HashMap<>();
         for (String project : projects.getClusterAliases()) {
             SearchResponse.Cluster computedProjectInfo = projects.getCluster(project);
             /*