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



##########
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:
       I was actually leaning towards a warning since the thread could sit here 
for a long time after only printing this message once.  But this could also be 
normal behavior in some situations so I settled on info. I don't think it 
should get printed a lot, unless something is wrong, hence why I was thinking 
of warning. This will only get printed if a tablet is being unloaded while also 
trying to be opened.




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