ctubbsii commented on a change in pull request #1881:
URL: https://github.com/apache/accumulo/pull/1881#discussion_r564850362
##########
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 Are you sure you're checking for the correct logger names?
The logger names should look like `tablet.X`.
I just quickly ran Uno and saw them with `grep '\[tablet[.][a-z]' *`
```
manager_localhost.log:2021-01-26T16:33:51,850 [tablet.location] DEBUG:
Assigned +r<< to localhost:9997[100032100b90004]
manager_localhost.log:2021-01-26T16:33:54,498 [tablet.location] DEBUG:
Assigned !0;~< to localhost:9997[100032100b90004]
...
tserver_localhost.log:2021-01-26T16:34:21,075 [tablet.files] DEBUG: Flushed
!0;~< created hdfs://localhost:8020/accumulo/tables/!0/table_info/F0000000.rf
from [memory]
tserver_localhost.log:2021-01-26T16:34:21,271 [tablet.files] DEBUG: Flushed
+r<< created hdfs://localhost:8020/accumulo/tables/+r/root_tablet/F0000001.rf
from [memory]
```
Remember, the `TabletLogger` loggers don't use the class name for the
logger, they have custom logger names.
----------------------------------------------------------------
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]
