impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Henry Robinson (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-4613: Make sure timers are finished before sending report profile
Date Wed, 07 Dec 2016 03:37:03 GMT
Henry Robinson has uploaded a new patch set (#2).

Change subject: IMPALA-4613: Make sure timers are finished before sending report profile
......................................................................

IMPALA-4613: Make sure timers are finished before sending report profile

FragmentComplete() sends the last report profile. If a timer is still
running while the profile is sent, it may not be included in the final
report. This patch ensures that the timers are all out of scope before
FragmentComplete() gets called.

Change-Id: If841e8b0a1effdab4349f081535b785c8bf2af7a
---
M be/src/runtime/plan-fragment-executor.cc
1 file changed, 16 insertions(+), 12 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/94/5394/2
-- 
To view, visit http://gerrit.cloudera.org:8080/5394
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If841e8b0a1effdab4349f081535b785c8bf2af7a
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Henry Robinson <henry@cloudera.com>

Mime
View raw message