[GitHub] [accumulo] cshannon commented on pull request #3331: Reuse already read lastLocation information during location updates

["cshannon (via GitHub)" <gi...@apache.org>]

cshannon commented on PR #3331:
URL: https://github.com/apache/accumulo/pull/3331#issuecomment-1518695637

   @keith-turner - It seemed like the best thing to do here was to add a new field to the `Assignment` class to pass the previously read last location data if known. This was nice as I didn't have to change method signatures and also works for the case where a list of assignments are passed. I added a bunch of mock tests to verify that the new behavior is correct in terms of when the update will do a metadata read or skip it. I ran through the sunny tests and they all passed but I will also kick off a full IT build.


-- 
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-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org