keith-turner commented on code in PR #4133:
URL: https://github.com/apache/accumulo/pull/4133#discussion_r1450896486
##########
server/manager/src/main/java/org/apache/accumulo/manager/split/SplitTask.java:
##########
@@ -84,8 +84,8 @@ public void run() {
return;
}
- var fateInstanceType =
-
FateInstanceType.toType(context.getTableIdToNameMap().get(tablet.getTableId()));
+ var fateInstanceType = FateInstanceType
+
.fromNamespaceOrTableName(context.getTableIdToNameMap().get(tablet.getTableId()));
Review Comment:
It would be outside the scope of this PR, but could address the drift issue
by creating a new internal AccumuoTables class and move the RootTable,
MetadataTable, and FateTable classes into it. Then it could have an
`Set<TableId> allIds()` method.
--
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]
