keith-turner commented on code in PR #5191:
URL: https://github.com/apache/accumulo/pull/5191#discussion_r1992313783
##########
server/tserver/src/main/java/org/apache/accumulo/tserver/AssignmentHandler.java:
##########
@@ -154,6 +157,28 @@ public void run() {
throw new RuntimeException("Minor compaction after recovery fails for
" + extent);
}
+ final Location expectedLocation =
Location.future(this.server.getTabletSession());
+ try (Ample.ConditionalTabletsMutator mutator =
Review Comment:
Still need to take the code that sets the location that pull it here and
combine it with this. Following `TabletStateStore.setLocation` you through
multiple layers of abstraction, will eventually find the impl that can be moved
here. Then all that code and and the layers could be removed. This is the
only place that calls TabletStateStore.setLocation
--
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]
