Alexey Serbin has uploaded this change for review. ( 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 --- M src/kudu/tablet/tablet.cc M src/kudu/tablet/tablet.h 2 files changed, 20 insertions(+), 22 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/80/19280/1 -- 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: newchange Gerrit-Change-Id: Iaf204d50bb3ee470790cefae983e6fb7b5a21c98 Gerrit-Change-Number: 19280 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
