You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ab...@apache.org on 2018/03/29 11:43:08 UTC
lucene-solr:master: SOLR-12145: ConcurrentModificationException in
SimNodeStateProvider.getReplicaInfo.
Repository: lucene-solr
Updated Branches:
refs/heads/master d7197b256 -> 668b81721
SOLR-12145: ConcurrentModificationException in SimNodeStateProvider.getReplicaInfo.
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/668b8172
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/668b8172
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/668b8172
Branch: refs/heads/master
Commit: 668b81721fa5b539d9286ed2f464426a598c352a
Parents: d7197b2
Author: Andrzej Bialecki <ab...@apache.org>
Authored: Thu Mar 29 13:42:19 2018 +0200
Committer: Andrzej Bialecki <ab...@apache.org>
Committed: Thu Mar 29 13:42:19 2018 +0200
----------------------------------------------------------------------
.../solr/cloud/autoscaling/sim/SimClusterStateProvider.java | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/668b8172/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimClusterStateProvider.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimClusterStateProvider.java b/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimClusterStateProvider.java
index 9a11a80..f6762fc 100644
--- a/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimClusterStateProvider.java
+++ b/solr/core/src/test/org/apache/solr/cloud/autoscaling/sim/SimClusterStateProvider.java
@@ -20,6 +20,7 @@ package org.apache.solr.cloud.autoscaling.sim;
import java.io.IOException;
import java.lang.invoke.MethodHandles;
import java.util.ArrayList;
+import java.util.Arrays;
import java.util.Collection;
import java.util.Collections;
import java.util.HashMap;
@@ -1155,14 +1156,15 @@ public class SimClusterStateProvider implements ClusterStateProvider {
/**
* Return all replica infos for a node.
* @param node node id
- * @return list of replicas on that node, or empty list if none
+ * @return copy of the list of replicas on that node, or empty list if none
*/
public List<ReplicaInfo> simGetReplicaInfos(String node) {
List<ReplicaInfo> replicas = nodeReplicaMap.get(node);
if (replicas == null) {
return Collections.emptyList();
} else {
- return replicas;
+ // make a defensive copy to avoid ConcurrentModificationException
+ return Arrays.asList(replicas.toArray(new ReplicaInfo[0]));
}
}