dlmarion commented on code in PR #3896:
URL: https://github.com/apache/accumulo/pull/3896#discussion_r1373750003
##########
server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java:
##########
@@ -407,6 +407,60 @@ private TableMgmtStats
manageTablets(Iterator<TabletManagement> iter,
state, goal, actions);
}
+ // If the current tablet is hosted and the TabletServer that it's hosted
on is being shut
+ // down, then we need to unhost tablets in order. We can't unassign the
root tablet before
+ // moving any metadata table tablets that are on this host. We can't
unassign metadata
+ // tablets before moving any user table tablets that are on this host.
+ if (current != null && state == TabletState.HOSTED
Review Comment:
Yeah, I wasn't sure about the user tables, but I followed the same pattern.
For sure, the metadata tablets have to be moved **before** the root tablet.
I'll try your suggestion regarding the user tablets.
--
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]
