You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "David Ribeiro Alves (Code Review)" <ge...@ip-10-146-233-104.ec2.internal> on 2016/02/05 05:38:35 UTC

[kudu-CR] [c++ client] - Remove KuduSchema from KuduScanBatch::RowPtr

David Ribeiro Alves has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/2065

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/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2065
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8aff27ac92babf4dc8b917cbe01c04e23b15de2a
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <da...@cloudera.com>