Csaba Ringhofer has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/18385


Change subject: IMPALA-11200: Avoid redundant "Codegen enabled" messages in 
profile
......................................................................

IMPALA-11200: Avoid redundant "Codegen enabled" messages in profile

Before this patch the message was added to the profile in Open(),
which can be called multiple times in subplans.

Moved it to Close(), which is only called once in the lifetime
of a Node/Aggregator. This can be also useful if async_codegen=1,
as the codegen may not be finished yet during Open().

A drawback of this is that this info won't be visible when the
Node is still active, but I don't think that it is a very useful
info in a still running query.

Change-Id: Iad2e31900ee6d29385cc8adc6bbf067d91f6450f
---
M be/src/exec/aggregator.cc
M be/src/exec/exec-node.cc
2 files changed, 7 insertions(+), 4 deletions(-)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iad2e31900ee6d29385cc8adc6bbf067d91f6450f
Gerrit-Change-Number: 18385
Gerrit-PatchSet: 1
Gerrit-Owner: Csaba Ringhofer <csringho...@cloudera.com>

Reply via email to