Hello Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23599
to look at the new patch set (#2).
Change subject: [tests] remove last vestiges of Atomics from tests
......................................................................
[tests] remove last vestiges of Atomics from tests
This changelist replaces Chromium-based Atomic with std::atomic
everywhere in various tests except for Atomic-related tests in the
src/kudu/util directory itself.
As for the memory ordering of the corresponding atomic operations,
std::memory_order_relaxed is used to match the former memory ordering
MemoryOrder::kMemOrderNoBarrier() of the Atomic class most closely.
This changelist doesn't contain any functional modifications.
Change-Id: Id8ea318787232cb69be09571c91e12b994a8e5c6
---
M src/kudu/benchmarks/tpch/tpch_real_world.cc
M src/kudu/clock/hybrid_clock-test.cc
M src/kudu/integration-tests/create-table-itest.cc
M src/kudu/integration-tests/create-table-stress-test.cc
M src/kudu/integration-tests/master-stress-test.cc
M src/kudu/integration-tests/raft_consensus-itest.cc
M src/kudu/integration-tests/test_workload.cc
M src/kudu/integration-tests/test_workload.h
M src/kudu/integration-tests/ts_recovery-itest.cc
M src/kudu/master/master-test.cc
M src/kudu/tools/ksck_remote-test.cc
M src/kudu/util/rw_mutex-test.cc
M src/kudu/util/semaphore.h
M src/kudu/util/semaphore_macosx.cc
M src/kudu/util/striped64-test.cc
M src/kudu/util/trace-test.cc
16 files changed, 184 insertions(+), 163 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/99/23599/2
--
To view, visit http://gerrit.cloudera.org:8080/23599
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id8ea318787232cb69be09571c91e12b994a8e5c6
Gerrit-Change-Number: 23599
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
