Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14293 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/14293 Reviewed-by: Tim Armstrong <tarmstr...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M be/src/exec/blocking-join-node.cc M be/src/util/stopwatch.h 2 files changed, 21 insertions(+), 29 deletions(-) Approvals: Tim Armstrong: Looks good to me, approved Impala Public Jenkins: Verified -- 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: merged Gerrit-Change-Id: I004833681248de65ce709fc746269686507aba54 Gerrit-Change-Number: 14293 Gerrit-PatchSet: 4 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>