Volodymyr Verovkin has posted comments on this change. ( http://gerrit.cloudera.org:8080/14219 )
Change subject: KUDU-1561 Implemented operator->() in KuduScanBatch::const_iterator ...................................................................... Patch Set 1: > What happens to existing application code using KuduScanBatch::RowPtr > or KuduScanBatch::const_iterator pointers? Does the dereferencing > behavior change with the new overloads? It does not change or override current functionality. It just adds operator->() KuduScanBatch::RowPtr and KuduScanBatch::const_iterator pointers Dereferencing is covered in test. -- 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: comment Gerrit-Change-Id: Ib5ddd99073d6a93337c184bee8b930cabeeda145 Gerrit-Change-Number: 14219 Gerrit-PatchSet: 1 Gerrit-Owner: Volodymyr Verovkin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Volodymyr Verovkin <[email protected]> Gerrit-Comment-Date: Thu, 12 Sep 2019 18:25:13 +0000 Gerrit-HasComments: No
