Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/12783 )

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)
Reviewed-on: http://gerrit.cloudera.org:8080/12783
Reviewed-by: Andrew Wong <[email protected]>
---
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(-)

Approvals:
  Kudu Jenkins: Verified
  Andrew Wong: Looks good to me, approved

--
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: merged
Gerrit-Change-Id: Icb5c436c9433acd87c44c4d81982420f33ebb4a4
Gerrit-Change-Number: 12783
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Greg Solovyev <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley <[email protected]>

Reply via email to