epugh commented on code in PR #1256: URL: https://github.com/apache/solr/pull/1256#discussion_r1142530690
########## solr/core/src/test/org/apache/solr/cloud/ForceLeaderTest.java: ########## @@ -305,8 +306,17 @@ private void doForceLeader(String collectionName, String shard) throws IOException, SolrServerException { CollectionAdminRequest.ForceLeader forceLeader = CollectionAdminRequest.forceLeaderElection(collectionName, shard); + boolean shardLeadersOnly = random().nextBoolean(); + RandomizingCloudSolrClientBuilder builder = Review Comment: I wish we had a commumnity standard for when to use a var... or that `gw tidy` did it for me.. I went ahead and put one in. -- 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. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org