Hello Zoltan Martonka, Yifan Zhang, Kudu Jenkins, helifu, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22016
to look at the new patch set (#3).
Change subject: [tablet] STL atomics for live row count in MemRowSet
......................................................................
[tablet] STL atomics for live row count in MemRowSet
This patch switches the counter of live rows in MemRowSet from AtomicInt
std::atomic<uint64_t>. I also took the liberty of adding a comment on
the semantics of the live row counter and cleaning up the code a bit.
This patch doesn't contain any functional modifications.
Change-Id: I9b85fc07b4c81bc3b3cf736d7f14b424ceef81a4
---
M src/kudu/tablet/memrowset.cc
M src/kudu/tablet/memrowset.h
2 files changed, 26 insertions(+), 25 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/16/22016/3
--
To view, visit http://gerrit.cloudera.org:8080/22016
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9b85fc07b4c81bc3b3cf736d7f14b424ceef81a4
Gerrit-Change-Number: 22016
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
Gerrit-Reviewer: helifu <[email protected]>
