Alexey Serbin has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/16035 )
Change subject: [consensus] use std::atomic in raft_consensus.{cc,h}
......................................................................
[consensus] use std::atomic in raft_consensus.{cc,h}
There was a mix of std::atomic<> and AtomicBool and others in
raft_consensus.{cc,h}. This patch switches all instances all
atomic varilables there to std::atomic.
Change-Id: I14214ca486b86b241f1b6fcf8842867a6bf1459a
Reviewed-on: http://gerrit.cloudera.org:8080/16035
Tested-by: Kudu Jenkins
Reviewed-by: Grant Henke <[email protected]>
---
M src/kudu/consensus/raft_consensus.cc
M src/kudu/consensus/raft_consensus.h
2 files changed, 13 insertions(+), 12 deletions(-)
Approvals:
Kudu Jenkins: Verified
Grant Henke: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/16035
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I14214ca486b86b241f1b6fcf8842867a6bf1459a
Gerrit-Change-Number: 16035
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <[email protected]>
