Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-3107: Change DataStreamMgr::lock_ to mutex ......................................................................
IMPALA-3107: Change DataStreamMgr::lock_ to mutex DataStreamSender::Cancel() takes several other locks while holding DataStreamMgr::lock_, so it's not a good idea for other clients of the class to be spinning on such a widely-held lock waiting for Cancel() to come back. Change-Id: Ifa9825188c7b5bbc991650be8b13ffcc1e580412 Reviewed-on: http://gerrit.cloudera.org:8080/2433 Reviewed-by: Dan Hecht <[email protected]> Reviewed-by: Silvius Rus <[email protected]> Tested-by: Internal Jenkins --- M be/src/runtime/data-stream-mgr.cc M be/src/runtime/data-stream-mgr.h 2 files changed, 7 insertions(+), 7 deletions(-) Approvals: Internal Jenkins: Verified Silvius Rus: Looks good to me, approved Dan Hecht: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/2433 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifa9825188c7b5bbc991650be8b13ffcc1e580412 Gerrit-PatchSet: 2 Gerrit-Project: Impala Gerrit-Branch: cdh5-2.5.0_5.7.0 Gerrit-Owner: Henry Robinson <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Silvius Rus <[email protected]>
