Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9443 )
Change subject: IMPALA-6577: avoid slow Status constructor on expiration thread ...................................................................... IMPALA-6577: avoid slow Status constructor on expiration thread As described in the JIRA, the query expiration thread gets bogged down generated stack traces, leading to slow expiration. I fixed that particular location to use Status::Expected(). I also fixed other similar problems in ImpalaServer while I was here. I added some logging where there wasn't already some so that we're not losing useful log output. Testing: Ran private core build on CentOS 6 (where the bug was reproducing at a high rate). Looped the test on my system for several hours. Change-Id: I7946dc3464b11580f768acbd5909d7e5fcae3360 Reviewed-on: http://gerrit.cloudera.org:8080/9443 Reviewed-by: Sailesh Mukil <[email protected]> Tested-by: Impala Public Jenkins --- M be/src/service/impala-beeswax-server.cc M be/src/service/impala-hs2-server.cc M be/src/service/impala-server.cc 3 files changed, 53 insertions(+), 23 deletions(-) Approvals: Sailesh Mukil: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9443 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I7946dc3464b11580f768acbd5909d7e5fcae3360 Gerrit-Change-Number: 9443 Gerrit-PatchSet: 4 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
