sigram commented on a change in pull request #2199:
URL: https://github.com/apache/lucene-solr/pull/2199#discussion_r564426476
##########
File path:
solr/core/src/java/org/apache/solr/cluster/placement/plugins/AffinityPlacementFactory.java
##########
@@ -467,7 +569,7 @@ private void makePlacementDecisions(SolrCollection
solrCollection, String shardN
if (candidateAzEntries == null) {
// This can happen because not enough nodes for the placement
request or already too many nodes with replicas of
// the shard that can't accept new replicas or not enough nodes with
enough free disk space.
- throw new PlacementException("Not enough nodes to place " +
numReplicas + " replica(s) of type " + replicaType +
+ throw new PlacementException("Not enough eligible nodes to place " +
numReplicas + " replica(s) of type " + replicaType +
Review comment:
it's already handled in `filterNodesWithCollection`
----------------------------------------------------------------
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
