Apply suggestions from code review - add ids to descriptions

Co-authored-by: Keith Turner <kturner@apache.org>

Author: DomGarguilo

server/manager/src/main/java/org/apache/accumulo/manager/compaction/coordinator/commit/CommitCompaction.java

@@ -130,7 +130,7 @@ private TabletMetadata commitCompaction(ServerContext ctx, ExternalCompactionId
 
         tabletMutator.submit(
             tabletMetadata -> !tabletMetadata.getExternalCompactions().containsKey(ecid),
-            () -> "commit compaction");
+            () -> "commit compaction "+ecid);
 
         if (LOG.isDebugEnabled()) {
           LOG.debug("Compaction completed {} added {} removed {}", tablet.getExtent(), newDatafile,

server/manager/src/main/java/org/apache/accumulo/manager/tableOps/bulkVer2/CleanUpBulkImport.java

@@ -111,7 +111,7 @@ private static void removeBulkLoadEntries(Ample ample, TableId tableId, FateId f
                 tabletsMutator.mutateTablet(tablet.getExtent()).requireAbsentOperation();
             tablet.getLoaded().entrySet().stream().filter(entry -> entry.getValue().equals(fateId))
                 .map(Map.Entry::getKey).forEach(tabletMutator::deleteBulkFile);
-            tabletMutator.submit(tm -> false, () -> "remove bulk load entries");
+            tabletMutator.submit(tm -> false, () -> "remove bulk load entries "+fateId);
           }
         }
 

server/manager/src/main/java/org/apache/accumulo/manager/tableOps/bulkVer2/LoadFiles.java

@@ -251,7 +251,7 @@ void load(List<TabletMetadata> tablets, Files files) {
         Preconditions.checkState(
             loadingFiles.put(tablet.getExtent(), List.copyOf(filesToLoad.keySet())) == null);
 
-        tabletMutator.submit(tm -> false, () -> "bulk load files");
+        tabletMutator.submit(tm -> false, () -> "bulk load files "+fateId);
       }
     }
 

server/manager/src/main/java/org/apache/accumulo/manager/tableOps/delete/ReserveTablets.java

@@ -96,7 +96,7 @@ public long isReady(FateId fateId, Manager manager) throws Exception {
           // must wait for the tablet to have no location before proceeding to actually delete. See
           // the documentation about the opid column in the MetadataSchema class for more details.
           conditionalMutator.mutateTablet(tabletMeta.getExtent()).requireAbsentOperation()
-              .putOperation(opid).submit(tm -> opid.equals(tm.getOperationId()), () -> "put opid");
+              .putOperation(opid).submit(tm -> opid.equals(tm.getOperationId()), () -> "put opid "+opid);
           submitted++;
         }
       }

server/manager/src/main/java/org/apache/accumulo/manager/tableOps/merge/DeleteRows.java

@@ -192,7 +192,7 @@ private Optional<KeyExtent> deleteTabletFiles(Manager manager, FateId fateId) {
             .submit(
                 tm -> tm.getFiles().containsAll(filesToAddMap.keySet())
                     && Collections.disjoint(tm.getFiles(), filesToDelete),
-                () -> "delete tablet files");
+                () -> "delete tablet files (as part of merge or deleterow operation) "+fateId);
       }
 
       var results = tabletsMutator.process();