Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20798 )
Change subject: IMPALA-12632: Use Atomics for CpuUsageRatio counters ...................................................................... IMPALA-12632: Use Atomics for CpuUsageRatio counters Now that IMPALA-12614 is fixed we see another data race in TSAN builds. Fix this by using the same strategy as IMPALA-12614, and use AtomicInt32 for the CpuUsageRatios values. Testing: TSAN tests cover this case. Change-Id: I373cb8ae1a45e5ec07318ccb5870e65efc906cca Reviewed-on: http://gerrit.cloudera.org:8080/20798 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/runtime/query-state.cc M be/src/util/system-state-info-test.cc M be/src/util/system-state-info.cc M be/src/util/system-state-info.h 4 files changed, 19 insertions(+), 17 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/20798 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I373cb8ae1a45e5ec07318ccb5870e65efc906cca Gerrit-Change-Number: 20798 Gerrit-PatchSet: 5 Gerrit-Owner: Andrew Sherman <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
