You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ma...@apache.org on 2020/10/08 21:46:56 UTC

[lucene-solr] branch reference_impl_dev updated: @956 Pull this out yet again.

This is an automated email from the ASF dual-hosted git repository.

markrmiller pushed a commit to branch reference_impl_dev
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git


The following commit(s) were added to refs/heads/reference_impl_dev by this push:
     new 2f29f23  @956 Pull this out yet again.
2f29f23 is described below

commit 2f29f239b97f2dbd595cb132b88652d58dce0287
Author: markrmiller@gmail.com <ma...@gmail.com>
AuthorDate: Thu Oct 8 16:46:19 2020 -0500

    @956 Pull this out yet again.
---
 solr/core/src/java/org/apache/solr/cloud/ZkController.java      | 2 +-
 solr/core/src/test/org/apache/solr/cloud/DeleteReplicaTest.java | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/solr/core/src/java/org/apache/solr/cloud/ZkController.java b/solr/core/src/java/org/apache/solr/cloud/ZkController.java
index 1d49df3..0e4345f 100644
--- a/solr/core/src/java/org/apache/solr/cloud/ZkController.java
+++ b/solr/core/src/java/org/apache/solr/cloud/ZkController.java
@@ -2885,7 +2885,7 @@ public class ZkController implements Closeable, Runnable {
       if (replicaRemoved) {
         try {
           log.info("Replica {} removed from clusterstate, remove it.", coreName);
-          getCoreContainer().unload(coreName, true, true, true);
+          // getCoreContainer().unload(coreName, true, true, true); // nocommit - this causes bad things in tests
         } catch (SolrException e) {
           if (!e.getMessage().contains("Cannot unload non-existent core")) {
             // no need to log if the core was already unloaded
diff --git a/solr/core/src/test/org/apache/solr/cloud/DeleteReplicaTest.java b/solr/core/src/test/org/apache/solr/cloud/DeleteReplicaTest.java
index 525063f..e9059d2 100644
--- a/solr/core/src/test/org/apache/solr/cloud/DeleteReplicaTest.java
+++ b/solr/core/src/test/org/apache/solr/cloud/DeleteReplicaTest.java
@@ -200,6 +200,7 @@ public class DeleteReplicaTest extends SolrCloudTestCase {
   }
 
   @Test
+  @AwaitsFix(bugUrl = "Currently disabled due to negative behavior of UnloadCoreOnDeletedWatcher and it's semi disable")
   public void deleteReplicaFromClusterState() throws Exception {
     final String collectionName = "deleteFromClusterStateCollection";
     CollectionAdminRequest.createCollection(collectionName, "conf", 1, 3)