ctubbsii commented on a change in pull request #1881:
URL: https://github.com/apache/accumulo/pull/1881#discussion_r564825155



##########
File path: 
server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java
##########
@@ -2220,6 +2220,7 @@ public void run() {
       synchronized (openingTablets) {
         while (openingTablets.contains(extent)) {
           try {
+            log.info("Waiting for tablet {} to finish opening before 
unloading.", extent);

Review comment:
       @milleruntime I looked at all the current logging in 
UnloadTabletHandler, and they all look like unlikely edge cases. I wouldn't be 
surprised if we rarely saw logs from it. If you add new logging to it that 
should show up, and it still doesn't, then it might be a problem... or if you 
know for sure one of these edge cases happened but it didn't get logged.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to