Hello Csaba Ringhofer, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14293
to look at the new patch set (#2).
Change subject: IMPALA-2312: simplify stopwatch ElapsedTime()
......................................................................
IMPALA-2312: simplify stopwatch ElapsedTime()
The stopwatches had two subtly different ElapsedTime()
interfaces, one that included previous running time
and one that didn't. They do not appear to be both
needed. I looked through all the uses and in all
cases using the old TotalElapsedTime() in place of
ElapsedTime() is equivalent, either because the
stopwatch is stopped when ElapsedTime() is called
or because Stop() is never called.
Change-Id: I004833681248de65ce709fc746269686507aba54
---
M be/src/exec/blocking-join-node.cc
M be/src/util/stopwatch.h
2 files changed, 21 insertions(+), 29 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/93/14293/2
--
To view, visit http://gerrit.cloudera.org:8080/14293
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I004833681248de65ce709fc746269686507aba54
Gerrit-Change-Number: 14293
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
