milleruntime commented on a change in pull request #1881:
URL: https://github.com/apache/accumulo/pull/1881#discussion_r564836051
##########
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:
@ctubbsii I agree, with the exception of what gets logged using the new
`TabletLogger` Keith added in #1316 . But I am not seeing anything from the
`TabletLogger` and I am not sure why. Maybe there is something we have to do
with Log4j2 to include those logs? Here is the class:
https://github.com/apache/accumulo/blob/cccac86255cd2d0ecd49e791162ca5cdf07512b6/core/src/main/java/org/apache/accumulo/core/logging/TabletLogger.java
----------------------------------------------------------------
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]
