Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8455 )
Change subject: IMPALA-6144: PublishFilter() continues to run after query failure/cancellation ...................................................................... IMPALA-6144: PublishFilter() continues to run after query failure/cancellation The CoordinatorBackendState::PublishFilter() function does not check for query failure/cancellation. So if runtime filters are being published during/after a failure, they will not be cancelled and still be sent out which may take a while depending on the size of the cluster. Also, these functions could potentially hold very large amounts of untracked memory. This patch fixes it by checking for cancellation/failure in PublishFilter. Change-Id: I400456ad85adb9c23d2d432d772311fa4dcff2ed Reviewed-on: http://gerrit.cloudera.org:8080/8455 Reviewed-by: Bharath Vissapragada <bhara...@cloudera.com> Reviewed-by: Tim Armstrong <tarmstr...@cloudera.com> Tested-by: Impala Public Jenkins --- M be/src/runtime/coordinator-backend-state.cc M be/src/runtime/coordinator.cc 2 files changed, 9 insertions(+), 0 deletions(-) Approvals: Bharath Vissapragada: Looks good to me, but someone else must approve Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8455 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I400456ad85adb9c23d2d432d772311fa4dcff2ed Gerrit-Change-Number: 8455 Gerrit-PatchSet: 3 Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Bharath Vissapragada <bhara...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>