Hello Will Berkeley, Tidy Bot, Kudu Jenkins, 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 (#3).

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, 433 insertions(+), 124 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/19/12619/3
--
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: 3
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: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Will Berkeley <[email protected]>

Reply via email to