Hello Will Berkeley, Kudu Jenkins,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/12783
to review the following change.
Change subject: [master] introduce cache for location mapping assignments
......................................................................
[master] introduce cache for location mapping assignments
This changelist adds a very primitive cache for location assignments.
The cache does not prevent running multiple commands for the same
location key if an entry is not present in the cache.
A unit test is also added.
Change-Id: Icb5c436c9433acd87c44c4d81982420f33ebb4a4
Reviewed-on: http://gerrit.cloudera.org:8080/12634
Reviewed-by: Will Berkeley <[email protected]>
Tested-by: Kudu Jenkins
(cherry picked from commit ae6bbcaabd20955119f1d945d276589538dae928)
---
M src/kudu/master/CMakeLists.txt
A src/kudu/master/location_cache-test.cc
A src/kudu/master/location_cache.cc
A src/kudu/master/location_cache.h
4 files changed, 421 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/83/12783/1
--
To view, visit http://gerrit.cloudera.org:8080/12783
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.9.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icb5c436c9433acd87c44c4d81982420f33ebb4a4
Gerrit-Change-Number: 12783
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley <[email protected]>