Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2726
to look at the new patch set (#2).
Change subject: [c++-client] abstract scanner configuration
......................................................................
[c++-client] abstract scanner configuration
Unlike the Java client, the c++ client configures scan options directly on the
KuduScanner. This commit adds an internal class, ScanConfiguration, that takes
over responsibility for configuring scans from the KuduScanner. KuduScanner is
updated to hold a ScanConfiguration internally. The ScanConfiguration will be
used internally in the forthecoming scan token builder API.
Change-Id: Id3d5b27f4a0ae6cd1500f17f96b7043590affd92
---
M src/kudu/client/CMakeLists.txt
M src/kudu/client/client.cc
A src/kudu/client/scan_configuration.cc
A src/kudu/client/scan_configuration.h
M src/kudu/client/scan_predicate.h
M src/kudu/client/scanner-internal.cc
M src/kudu/client/scanner-internal.h
M src/kudu/client/schema.h
8 files changed, 434 insertions(+), 175 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/26/2726/2
--
To view, visit http://gerrit.cloudera.org:8080/2726
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id3d5b27f4a0ae6cd1500f17f96b7043590affd92
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
