Hello Will Berkeley, Tidy Bot, Andrew Wong, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12619
to look at the new patch set (#5).
Change subject: [master] cache assigned locations
......................................................................
[master] cache assigned locations
This changelist adds a very primitive cache to master to cache
the location once assigned to tablet servers and clients.
The cache does not prevent running multiple commands for the same
location key if an entry is not present in the cache.
The idea is that we should not expect ConnectToCluster requests
from the same node running concurrently.
More advanced cache that takes care of the thundering herd pattern
is in the works, but I decided to put up this simple one as a first
step.
Change-Id: I12c8952c43a8ad352acd46c8006824b2ad9d1204
---
M src/kudu/client/client-test.cc
M src/kudu/integration-tests/location_assignment-itest.cc
M src/kudu/master/CMakeLists.txt
A src/kudu/master/location_cache.cc
A src/kudu/master/location_cache.h
M src/kudu/master/master-test.cc
M src/kudu/master/master.cc
M src/kudu/master/master.h
M src/kudu/master/master_service.cc
M src/kudu/master/ts_descriptor-test.cc
M src/kudu/master/ts_descriptor.cc
M src/kudu/master/ts_descriptor.h
M src/kudu/master/ts_manager.cc
M src/kudu/master/ts_manager.h
M src/kudu/tools/ksck_remote-test.cc
15 files changed, 440 insertions(+), 128 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/19/12619/5
--
To view, visit http://gerrit.cloudera.org:8080/12619
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I12c8952c43a8ad352acd46c8006824b2ad9d1204
Gerrit-Change-Number: 12619
Gerrit-PatchSet: 5
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Will Berkeley <[email protected]>
