keith-turner opened a new pull request, #5308: URL: https://github.com/apache/accumulo/pull/5308
When many accumulo client threads all decided to invalidate some servers at the same time in the tablet location cache each threads would scan though all metadata looking for those servers. This change adds servers to a set for deferred processing when invalidated. This has two advantages. First for multiple threads adding servers at around the same time it can lower the number of scans done over all the cached tablet locations. Second the amount of work done to invalidate servers is lowered. The code used to find cache entries for a server, add the extent for that cache entry to badExtents, and then later look for everything in badExtents and remove it. This change removes all of that work and directly removes entries from the cache and avoids adding anything to badExtents. -- 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]
