DomGarguilo commented on PR #6027: URL: https://github.com/apache/accumulo/pull/6027#issuecomment-3672035280
> This looks good, was there anything else you were planning to do before taking out of draft? The main thing I wanted to look into was how we can improve things in `TabletInformationCollector.getTabletInformation()`. Right now we are looping over each range returned after merging the given range list and getting a new `TabletsMetadata` for each of those ranges. I think overall the new way of doing things here is an improvement but it would be really nice if we could get a single `TabletsMetadata` for that whole list of Ranges. This isn't possible right now and we would need to add a new builder option that accepts multiple Ranges. Maybe this shouldn't block things here though since this new code is already an improvement and we can look into adding that new feature in a follow on. -- 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]
