Thomas Tauber-Marshall has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16835
Change subject: IMPALA-10383: Fix data race on AdmissionController::queue_nodes_ ...................................................................... IMPALA-10383: Fix data race on AdmissionController::queue_nodes_ IMPALA-9930 introduces a new data structure to AdmissionController: 'queue_nodes_'. The intention was that access to 'queue_nodes_' should always be protected by 'queue_nodes_lock_', but this was not always the case. This patch fixes the issue by ensuring that 'queue_nodes_lock_' is held everywhere that 'queue_nodes_' is used. Testing: - Ran TSAN and confirmed there is no longer a data race being reported. Change-Id: I05b70112300d3a886bdd5f02fda0bd7ca98df6cf --- M be/src/scheduling/admission-controller.cc 1 file changed, 8 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/35/16835/1 -- To view, visit http://gerrit.cloudera.org:8080/16835 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I05b70112300d3a886bdd5f02fda0bd7ca98df6cf Gerrit-Change-Number: 16835 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Tauber-Marshall <[email protected]>
