dlmarion commented on code in PR #5811:
URL: https://github.com/apache/accumulo/pull/5811#discussion_r2290891456


##########
server/base/src/main/java/org/apache/accumulo/server/ServerContext.java:
##########
@@ -522,9 +522,11 @@ public void close() {
       getMetricsInfo().close();
     }
     if (sharedSchedExecutorCreated.get()) {
+      log.info("Shutting down shared executor pool");

Review Comment:
   Changed to debug in d41dbcc



##########
core/src/main/java/org/apache/accumulo/core/clientImpl/ClientContext.java:
##########
@@ -918,21 +918,26 @@ public AuthenticationToken token() {
   @Override
   public synchronized void close() {
     if (closed.compareAndSet(false, true)) {
-      if (zooCacheCreated.get()) {
-        zooCache.get().close();
-      }
-      if (zooKeeperOpened.get()) {
-        zooSession.get().close();
-      }
       if (thriftTransportPool != null) {
+        log.info("Closing Thrift Transport Pool");
         thriftTransportPool.shutdown();
       }
       if (scannerReadaheadPool != null) {
+        log.info("Closing Scanner ReadAhead Pool");
         scannerReadaheadPool.shutdownNow(); // abort all tasks, client is 
shutting down
       }
       if (cleanupThreadPool != null) {
+        log.info("Closing Cleanup ThreadPool");
         cleanupThreadPool.shutdown(); // wait for shutdown tasks to execute
       }
+      if (zooCacheCreated.get()) {
+        log.info("Closing ZooCache");

Review Comment:
   Changed to debug in d41dbcc



-- 
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: notifications-unsubscr...@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to