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



##########
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:
       If it can happen as a result of a user action that they have control 
over, then warning might make more sense. If it's something that happens, and 
is informative to all users, then info might make sense. If it's informative 
only for troubleshooting, then debug might make sense. I'm not sure which this 
falls in. I assumed the "only for troubleshooting" category, but I haven't 
dived in to form a more informed opinion.




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