keith-turner commented on code in PR #5993:
URL: https://github.com/apache/accumulo/pull/5993#discussion_r2578949222
##########
core/src/main/java/org/apache/accumulo/core/logging/TabletLogger.java:
##########
@@ -140,8 +141,8 @@ private static Collection<String>
asMinimalString(Collection<CompactableFile> fi
cf -> CompactableFileImpl.toStoredTabletFile(cf).toMinimalString());
}
- public static void renamed(KeyExtent extent, TabletFile src, TabletFile
dest) {
- fileLog.debug("{} renamed {} to {}", extent, src.getFileName(),
dest.getFileName());
+ public static void renamed(TableId tableId, Path src, Path dest) {
Review Comment:
Created #5996 based on trying to make bulk v2 easier to follow in the logs.
The rename logging could go there. The TabletLogger class tracks operations
related to an individual tablet. The moves related to a bulk import are not
scoped to any single tablet, they are most tightly scoped to a fate operation
id and more loosely scoped to a table id.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
