David Ribeiro Alves has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/2065 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> --- 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(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2065 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8aff27ac92babf4dc8b917cbe01c04e23b15de2a 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: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
