David Ribeiro Alves has submitted this change and it was merged.

Change subject: [c++ client] - Expose the schema being scanned in KuduScanBatch
......................................................................


[c++ client] - Expose the schema being scanned in KuduScanBatch

This exposes the schema being scanned in the KuduScanBatch and 
KuduScanBatch::RowPtr
allowing to pass pointer/references to these around when in use without an 
accompanying
schema, which was hard to infer in any case since the scanner projection is set 
with
just column names.

Note that this is just exposing the schema information that was already there, 
just
in KuduSchema form (vs. the internal Schema) which is more suited for user 
consumption.

Change-Id: Id4276036d5ba7faf7959b4324fd013b68a08bec5
Reviewed-on: http://gerrit.cloudera.org:8080/2020
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <[email protected]>
---
M src/kudu/client/client.cc
M src/kudu/client/scan_batch.cc
M src/kudu/client/scan_batch.h
M src/kudu/client/scanner-internal.cc
M src/kudu/client/scanner-internal.h
M src/kudu/client/schema.h
M src/kudu/tools/ts-cli.cc
7 files changed, 57 insertions(+), 8 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/2020
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id4276036d5ba7faf7959b4324fd013b68a08bec5
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to