Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10716 )
Change subject: Add missing PrintId() calls ...................................................................... Add missing PrintId() calls For consistency, add PrintId() around query_id() used in a stream, which was missing from this commit but needed for 2.x backport: IMPALA-5216: Make admission control queuing async This change was put into the cherry-pick for 2.x, so: Cherry-picks: not for 2.x Change-Id: Ifa53bf46411b09f33ccf38b080cd1c06dd3717ef Reviewed-on: http://gerrit.cloudera.org:8080/10716 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/scheduling/admission-controller.cc M be/src/service/client-request-state.cc 2 files changed, 7 insertions(+), 5 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/10716 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ifa53bf46411b09f33ccf38b080cd1c06dd3717ef Gerrit-Change-Number: 10716 Gerrit-PatchSet: 3 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]>
