Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Adar Dembo,

I'd like you to reexamine a change. Please visit

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

to look at the new patch set (#2).

Change subject: KUDU-1561 Implemented operator->() in 
KuduScanBatch::const_iterator
......................................................................

KUDU-1561 Implemented operator->() in KuduScanBatch::const_iterator

operator->() requires returning pointer or object which overloads access
via pointer operator (operator->() again).
Since KuduScanBatch::const_iterator does to keep current
KuduScanBatch::RowPtr, we cannot return pointer to it in operator ->.
Instead, we return KuduScanBatch::RowPtr which implements pointer trait
(operator->).

Change-Id: Ib5ddd99073d6a93337c184bee8b930cabeeda145
---
M src/kudu/client/client-test.cc
M src/kudu/client/scan_batch.h
2 files changed, 28 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/19/14219/2
--
To view, visit http://gerrit.cloudera.org:8080/14219
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib5ddd99073d6a93337c184bee8b930cabeeda145
Gerrit-Change-Number: 14219
Gerrit-PatchSet: 2
Gerrit-Owner: Volodymyr Verovkin <verjov...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Volodymyr Verovkin <verjov...@cloudera.com>

Reply via email to