Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23221 )
Change subject: [server] remove last vestiges of Atomics from server ...................................................................... [server] remove last vestiges of Atomics from server This patch switches 'mem_tracker_id_counter' from Chromium-based Atomic to std::atomic. Also, the 'mem_tracker_id_counter' static variable moved inside of the CreateMemTrackerForServer() function to avoid warnings on non-POD global statics. This changelist doesn't contain any functional modifications. Change-Id: I9ddfd14b94a8c5e823a5f9394c8eaed4208591d2 Reviewed-on: http://gerrit.cloudera.org:8080/23221 Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Alexey Serbin <[email protected]> --- M src/kudu/server/server_base.cc 1 file changed, 6 insertions(+), 7 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/23221 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I9ddfd14b94a8c5e823a5f9394c8eaed4208591d2 Gerrit-Change-Number: 23221 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
