You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2019/09/11 16:18:03 UTC
[GitHub] [accumulo] keith-turner opened a new pull request #1356: Use
concurrent map instead of cache for bulkImported map
keith-turner opened a new pull request #1356: Use concurrent map instead of cache for bulkImported map
URL: https://github.com/apache/accumulo/pull/1356
While analyzing some tablet code I noticed a Guava cahce was being used
like a map. When I first saw the code I was concerned the cache may
evict entries (which would be a bug). However the cache was created
with default settings, which according to the javadoc does not
automatically evict. This commit replaces the cache with a concurrent
map so no one has to do this analysis again.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services