[GitHub] [accumulo] keith-turner commented on pull request #3324: Rename TabletLocator to TabletCache

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner commented on PR #3324:
URL: https://github.com/apache/accumulo/pull/3324#issuecomment-1516613217

   > I have no issue with merging with TabletCache, but I wonder if TabletLocationCache is a better name. Not that TabletCache is a bad name, but there are other classes (e.g. MetadataLocationObtainer) and variables (e.g. locator) that imply location.
   
   I don't like location in the name since the location is optional.  Was thinking the name needed more specificity though after reading the comment,  changed the name to ClientTabletCache to make it more specific.  I realized I had another todo in the code for doing a rename, so I did that.  I also renamed MetadataLocationObtainer.


-- 
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