Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/3581 to look at the new patch set (#3). Change subject: [c++-client]: cache non-covering ranges in meta cache ...................................................................... [c++-client]: cache non-covering ranges in meta cache This commit introduces a few features to the meta cache, all with the aim of making it compatible with the upcoming add/drop range partitions feature. 1) Non-covered range partitions are now cached in the meta cache. This is achieved by storing MetaCacheEntry objects in the meta cache's partition-key index instead of RemoteTablets. The MetaCacheEntry holds either a RemoteTablet, in which case it represents a covered partition range, or it represents a non-covered partition range. 2) Entries are now removed from the meta cache's partition-key index when it can be determined that the entries are no longer valid from the results of a GetTableLocations RPC. 3) A basic TTL has been added to the GetTableLocationsResponsePB so that the client can properly refresh the meta cache when necessary. Follow up commits will expand on this feature and make it configurable, perhaps on a per-table basis. Change-Id: I05bcb3fe05d51d7c455e1d68bd2baa6f3c2b9d21 --- M src/kudu/client/meta_cache.cc M src/kudu/client/meta_cache.h M src/kudu/client/schema.h M src/kudu/master/master.proto 4 files changed, 278 insertions(+), 90 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/81/3581/3 -- To view, visit http://gerrit.cloudera.org:8080/3581 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I05bcb3fe05d51d7c455e1d68bd2baa6f3c2b9d21 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>