You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "David Ribeiro Alves (Code Review)" <ge...@cloudera.org> on 2016/05/11 21:59:06 UTC

[kudu-CR] Move the MetaCacheReplicaPicker into meta_cache.h/cc and make it a member of RemoteTablet

David Ribeiro Alves has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/3028

Change subject: Move the MetaCacheReplicaPicker into meta_cache.h/cc and make it a member of RemoteTablet
......................................................................

Move the MetaCacheReplicaPicker into meta_cache.h/cc and make it a member of RemoteTablet

This moves the MetaCacheReplicaPicker into meta_cache.h/cc and makes it so
that one instance of the picked is shared by all users of a certain RemoteTablet.

The end result is that further improvement of the replica picker (for instance
to make it more of a part of RemoteTablet with a less redundant code path)
can be isolated from other components. batcher.cc no longer contains any
reference to the replica picking code.

Change-Id: I5344e9556f7042743458efbf1a5dc8be1e45f8da
---
M src/kudu/client/batcher.cc
M src/kudu/client/meta_cache.cc
M src/kudu/client/meta_cache.h
3 files changed, 225 insertions(+), 187 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/28/3028/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3028
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5344e9556f7042743458efbf1a5dc8be1e45f8da
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <da...@cloudera.com>