keith-turner commented on code in PR #4433:
URL: https://github.com/apache/accumulo/pull/4433#discussion_r1554210556
##########
server/manager/src/main/java/org/apache/accumulo/manager/ManagerClientServiceHandler.java:
##########
@@ -648,9 +649,12 @@ public void requestTabletHosting(TInfo tinfo, TCredentials
credentials, String t
inProgress.forEach(hostingRequestInProgress::remove);
}
- // ELASTICITY_TODO pass ranges of individual tablets
- manager.getEventCoordinator().event(success, "Tablet hosting requested for
%d tablets in %s",
- success.size(), tableId);
+ final List<Range> ranges = new ArrayList<>(success.size());
+ success.forEach(ke -> ranges.add(ke.toDataRange()));
+ final List<Range> tabletRanges = Range.mergeOverlapping(ranges);
+ manager.getEventCoordinator().event(success,
+ "Tablet hosting requested for %d tablets in ranges %s in table %s",
success.size(),
+ tabletRanges, tableId);
Review Comment:
Also, there is existing logging earlier in the method for each tablet that
needs to be hosted.
--
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
