milleruntime commented on a change in pull request #1881:
URL: https://github.com/apache/accumulo/pull/1881#discussion_r564815966
##########
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 am working on a change in this code for 2.1 and it includes a lot more
logging, like this one and they all seem to be more appropriate at debug. I
can make it debug since its really out of user control.
I have noticed something strange with this class logging in 2.1 that I can't
seem to figure out. I don't see any of the logging for the UnloadTabletHandler
in 2.1 (using Uno) not even any of the INFO logs. I wonder if its because the
UnloadTabletHandler got moved out of TabletServer class when the
ThriftClientHandler was moved out in #1584. I also wonder if any of the other
classes in that change were affected.
----------------------------------------------------------------
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]
