Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18724 )
Change subject: [refactor] Fix some lint and tidy errors ...................................................................... [refactor] Fix some lint and tidy errors This patch has no functional changes, it fixes some lint and tidy errors, it's the first step to implement immutable column feature, aiming to reduce the diff code when we review that patch in https://gerrit.cloudera.org/c/18241/. Change-Id: I89b0f3b9ace02f380ce1193bc1facf2aaf135096 Reviewed-on: http://gerrit.cloudera.org:8080/18724 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/client/write_op.h M src/kudu/common/row.h M src/kudu/common/row_changelist.cc M src/kudu/common/row_changelist.h M src/kudu/common/schema-test.cc M src/kudu/tablet/delta_applier.cc M src/kudu/tablet/delta_applier.h M src/kudu/tablet/delta_iterator_merger.cc M src/kudu/tablet/delta_iterator_merger.h M src/kudu/tablet/delta_store.h M src/kudu/tablet/deltafile.cc M src/kudu/tablet/deltafile.h M src/kudu/tablet/deltamemstore-test.cc M src/kudu/tablet/deltamemstore.cc M src/kudu/tablet/deltamemstore.h M src/kudu/tablet/ops/op.h M src/kudu/tablet/rowset.cc M src/kudu/tablet/rowset.h M src/kudu/tablet/tablet-decoder-eval-test.cc M src/kudu/tablet/tablet.cc 20 files changed, 78 insertions(+), 89 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/18724 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I89b0f3b9ace02f380ce1193bc1facf2aaf135096 Gerrit-Change-Number: 18724 Gerrit-PatchSet: 3 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
