Yifan Zhang has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19280 )
Change subject: [tablet] clean-up on Tablet::UpdateLastReadTime() ...................................................................... [tablet] clean-up on Tablet::UpdateLastReadTime() Tablet::UpdateLastReadTime() was marked 'const' while it wasn't so semantically, but the comment in the header file was claiming it's necessary. However, it turned out that's not so as of now, and this patch reduces the confusion. I also took the liberty to make formatting changes in the constructor to comply with the C++ code style guide. This patch doesn't contain any functional changes. This is a follow-up to ca957fb86736b1e07b18d80fdfa4e41c191e7072. Change-Id: Iaf204d50bb3ee470790cefae983e6fb7b5a21c98 Reviewed-on: http://gerrit.cloudera.org:8080/19280 Tested-by: Kudu Jenkins Reviewed-by: Yifan Zhang <[email protected]> --- M src/kudu/tablet/tablet.cc M src/kudu/tablet/tablet.h 2 files changed, 20 insertions(+), 22 deletions(-) Approvals: Kudu Jenkins: Verified Yifan Zhang: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/19280 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iaf204d50bb3ee470790cefae983e6fb7b5a21c98 Gerrit-Change-Number: 19280 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
