Jason Fehr has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/21744


Change subject: IMPALA-13350: Fixes TSAN Thread Leak Caused Workload Management 
Thread
......................................................................

IMPALA-13350: Fixes TSAN Thread Leak Caused Workload Management Thread

The workload management processing runs in a separate thread declared
in impala-server.h.  This thread runs until a graceful shutdown is
intiated.  The last step of the Impala coordinator shutdown process
is to drain the completed queries queue to the query log table thus
ensuring completed queries do not get lost.

This thread has to run to completion, but the coordinator shutdown
process never joins that thread. This patch adds the joining of that
thread during the coordinator shutdown process.  If the workload
management shutdown process exceedes the allotted time, the thread is
detached.

Change-Id: I1e95967bb6e04470a8900c9ba69080eea8aaa25e
---
M be/src/service/impala-server.cc
M be/src/service/impala-server.h
M be/src/service/workload-management-init.cc
M be/src/service/workload-management.cc
M be/src/service/workload-management.h
5 files changed, 41 insertions(+), 12 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/44/21744/1
--
To view, visit http://gerrit.cloudera.org:8080/21744
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I1e95967bb6e04470a8900c9ba69080eea8aaa25e
Gerrit-Change-Number: 21744
Gerrit-PatchSet: 1
Gerrit-Owner: Jason Fehr <[email protected]>

Reply via email to