kevinrr888 commented on code in PR #5416:
URL: https://github.com/apache/accumulo/pull/5416#discussion_r2056562348
##########
server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java:
##########
@@ -644,9 +644,7 @@ private TableMgmtStats
manageTablets(Iterator<TabletManagement> iter,
}
switch (state) {
case HOSTED:
- if
(location.getServerInstance().equals(manager.migrations.get(tm.getExtent()))) {
- manager.migrations.remove(tm.getExtent());
- }
+ manager.conditionallyDeleteMigration(tm.getExtent(),
location.getServerInstance());
Review Comment:
Dropped this call to conditionallyDeleteMigration and Migration cleanup code
now checks for this case 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
