WIP

server/tserver/src/main/java/org/apache/accumulo/tserver/TabletServer.java

@@ -1092,7 +1092,8 @@ public boolean needsRecovery(TabletMetadata tabletMetadata){
     }
 
     try {
-      // TODO cache finished marker
+      // TODO cache finished marker.. or maybe cache the child dirs
+      // TODO the code inside logger seems to be looking for the finish marker also
       List<Path> recoveryDirs = new ArrayList<>();
       for (LogEntry entry : logEntries) {
         Path recovery = null;
@@ -1109,7 +1110,11 @@ public boolean needsRecovery(TabletMetadata tabletMetadata){
         recoveryDirs.add(recovery);
       }
 
-      return logger.needsRecovery(getContext(), tabletMetadata.getExtent(), recoveryDirs, resourceManager.getFileLenCache(), new BasicCacheProvider(resourceManager.getIndexCache(), resourceManager.getDataCache()));
+      boolean needsRecovery =  logger.needsRecovery(getContext(), tabletMetadata.getExtent(), recoveryDirs, resourceManager.getFileLenCache(), new BasicCacheProvider(resourceManager.getIndexCache(), resourceManager.getDataCache()));
+      if(!needsRecovery) {
+        // TODO this may be redundant , look at the over logs
+        log.debug("tablet {} has walogs but does not have any data in them", tabletMetadata.getExtent());
+      }
     }catch (IOException e){
       throw new UncheckedIOException(e);
     }