Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2065
to look at the new patch set (#2).
Change subject: [c++ client] - Remove KuduSchema from KuduScanBatch::RowPtr
......................................................................
[c++ client] - Remove KuduSchema from KuduScanBatch::RowPtr
Todd manifested concern that this was making the pointer less like a pure
pointer, i.e. more bloated.
The main reason this was made available here at all was that the python
client was still using KuduRowResult and thus needed it. Since that is
no longer the case this field (and its accessors) can be removed.
Change-Id: I8aff27ac92babf4dc8b917cbe01c04e23b15de2a
---
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
4 files changed, 2 insertions(+), 11 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/65/2065/2
--
To view, visit http://gerrit.cloudera.org:8080/2065
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8aff27ac92babf4dc8b917cbe01c04e23b15de2a
Gerrit-PatchSet: 2
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]>