kevinrr888 commented on code in PR #5416:
URL: https://github.com/apache/accumulo/pull/5416#discussion_r2056568128
##########
server/manager/src/main/java/org/apache/accumulo/manager/Manager.java:
##########
@@ -530,8 +527,13 @@ ManagerGoalState getManagerGoalState() {
}
public void clearMigrations(TableId tableId) {
- synchronized (migrations) {
- migrations.keySet().removeIf(extent -> extent.tableId().equals(tableId));
+ var ample = getContext().getAmple();
+ // prev row needed for the extent
+ try (var tabletsMetadata = ample.readTablets().forTable(tableId)
+ .fetch(TabletMetadata.ColumnType.PREV_ROW,
TabletMetadata.ColumnType.MIGRATION).build()) {
+ for (TabletMetadata tabletMetadata : tabletsMetadata) {
+
ample.mutateTablet(tabletMetadata.getExtent()).deleteMigration().mutate();
Review Comment:
Good call, I hadn't looked into what operation the CleanUp repo was a part
of. Dropped method in 0a9455cdea88f167179953a1eb8073bf7a77c2f3
--
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: notifications-unsubscr...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
