[lucene-solr] branch reference_impl updated: @311 Hmm, can't respond to close like this, it comes after connected even when close was not called ... @302 Play with shutdown hooks.

[ma...@apache.org]

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

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


The following commit(s) were added to refs/heads/reference_impl by this push:
     new 71859d2  @311 Hmm, can't respond to close like this, it comes after connected even when close was not called ... @302 Play with shutdown hooks.
71859d2 is described below

commit 71859d2c89f1279b0ecf014216444db0b679b3f0
Author: markrmiller@gmail.com <ma...@gmail.com>
AuthorDate: Wed Jul 22 15:27:14 2020 -0500

    @311 Hmm, can't respond to close like this, it comes after connected even when close was not called ... @302 Play with shutdown hooks.
---
 .../src/java/org/apache/solr/common/cloud/ConnectionManager.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/solr/solrj/src/java/org/apache/solr/common/cloud/ConnectionManager.java b/solr/solrj/src/java/org/apache/solr/common/cloud/ConnectionManager.java
index 2862485..0a97529 100644
--- a/solr/solrj/src/java/org/apache/solr/common/cloud/ConnectionManager.java
+++ b/solr/solrj/src/java/org/apache/solr/common/cloud/ConnectionManager.java
@@ -265,9 +265,9 @@ public class ConnectionManager implements Watcher, Closeable {
       disconnected();
       connectionStrategy.disconnected();
     } else if (state == KeeperState.Closed) {
-      log.info("zkClient has disconnected");
-      disconnected();
-      connectionStrategy.disconnected();
+      log.info("zkClient state == closed");
+      //disconnected();
+      //connectionStrategy.disconnected();
     } else if (state == KeeperState.AuthFailed) {
       log.warn("zkClient received AuthFailed");
     }