commit ed97eac76da47f711f8f901810cc9d4d8fb27214
author Mike Miller <mmiller@apache.org> Wed Jan 27 08:09:54 2021 -0500
committer GitHub <noreply@github.com> Wed Jan 27 08:09:54 2021 -0500
tree 6f296ac1cb246cbe1508df987fccd630bd665b3e
parent 48e483b6fab9006a224b4630dfdc3449110cb9a0

Make UnloadTabletHandler log before waiting (#1881)

* If a tablet is in the process of being opened, the UnloadTabletHandler
will wait for a tablet to be removed from the list before proceeding
with the unload.  This could take a long time so log info before waiting.
* Fix Typo in UnloadTabletHandler

server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java

diff --git a/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java b/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
index 73bc58f..e141de9 100644
--- a/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
+++ b/server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
@@ -2220,6 +2220,7 @@
       synchronized (openingTablets) {
         while (openingTablets.contains(extent)) {
           try {
+            log.info("Waiting for tablet {} to finish opening before unloading.", extent);
             openingTablets.wait();
           } catch (InterruptedException e) {}
         }
@@ -2246,7 +2247,7 @@
       } catch (Throwable e) {
 
         if ((t.isClosing() || t.isClosed()) && e instanceof IllegalStateException) {
-          log.debug("Failed to unload tablet {} ... it was alread closing or closed : {}", extent,
+          log.debug("Failed to unload tablet {} ... it was already closing or closed : {}", extent,
               e.getMessage());
         } else {
           log.error("Failed to close tablet {}... Aborting migration", extent, e);