You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Dan Burkert (Code Review)" <ge...@cloudera.org> on 2016/05/23 21:55:19 UTC

[kudu-CR] [c++-client]: minimal changes to support tables with non-covering range partitions

Hello Jean-Daniel Cryans, Todd Lipcon,

I'd like you to do a code review.  Please visit

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

to review the following change.

Change subject: [c++-client]: minimal changes to support tables with non-covering range partitions
......................................................................

[c++-client]: minimal changes to support tables with non-covering range partitions

This commit introduces the minimum set of changes necessary to add support to
the C++ client for normal operations (open/write/scan) on tables with
non-covering range partitions. Tests are not included, since the ability to
create such tables has not been introduced in the master or client. These
changes have been tested by hand, and in tests included in a followup commit to
add create table with range bounds support. This minimal functionality is being
pushed earlier in order to allow the 0.9 client to be compatible with 0.10
servers which have non-covered range partitioned tables.

Change-Id: Ib25b7a57b14b3d1e4e6dca75b88dad7c19ba7565
---
M src/kudu/client/meta_cache.cc
M src/kudu/client/meta_cache.h
M src/kudu/client/scan_token-internal.cc
M src/kudu/client/scanner-internal.cc
M src/kudu/client/scanner-internal.h
M src/kudu/common/partition_pruner-test.cc
M src/kudu/common/partition_pruner.cc
M src/kudu/common/partition_pruner.h
8 files changed, 123 insertions(+), 36 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib25b7a57b14b3d1e4e6dca75b88dad7c19ba7565
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <da...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>