Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3017
to look at the new patch set (#5).
Change subject: Add an all-virtual ReplicaPicker class and a meta cache backed
implementation for replicated rpcs
......................................................................
Add an all-virtual ReplicaPicker class and a meta cache backed implementation
for replicated rpcs
This pulls out the replica picking code from WriteRpc and into an implementation
of the new ReplicaPicker all-virtual class.
The new replica picker implementation, which is backed by the meta cache, fully
readies the replica before handing it hover to WriteRpc. This makes the writing
logic simples (less callbacks) and makes the replica picking code possibly
reusable.
Again, this is just a refactor of existing code so no new tests were added.
Change-Id: Id43db316606cb807ec0019c79b3bdf76fa509fe5
---
M src/kudu/client/batcher.cc
M src/kudu/rpc/replicated_rpc.h
2 files changed, 258 insertions(+), 149 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/17/3017/5
--
To view, visit http://gerrit.cloudera.org:8080/3017
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id43db316606cb807ec0019c79b3bdf76fa509fe5
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
