Dan Burkert has submitted this change and it was merged. 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 forthcoming scan token builder API. Change-Id: Id3d5b27f4a0ae6cd1500f17f96b7043590affd92 Reviewed-on: http://gerrit.cloudera.org:8080/2726 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- 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, 428 insertions(+), 171 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2726 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id3d5b27f4a0ae6cd1500f17f96b7043590affd92 Gerrit-PatchSet: 7 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
