Hello Tim Armstrong, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15285
to look at the new patch set (#2).
Change subject: IMPALA-4080 [part 4]: Moved code responsible for codegen from
select and union's exec node to their plan node.
......................................................................
IMPALA-4080 [part 4]: Moved code responsible for codegen from select
and union's exec node to their plan node.
Refactored code responsible for codegen from select and union's exec
node to their plan node.
Testing:
Successfully ran exhaustive tests.
Manually verified that codegen works for all modified exec nodes.
Change-Id: I0610b93b9a13da19e35151b231349c0ab16071d2
---
M be/src/exec/select-node.cc
M be/src/exec/select-node.h
M be/src/exec/union-node.cc
M be/src/exec/union-node.h
4 files changed, 76 insertions(+), 37 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/85/15285/2
--
To view, visit http://gerrit.cloudera.org:8080/15285
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I0610b93b9a13da19e35151b231349c0ab16071d2
Gerrit-Change-Number: 15285
Gerrit-PatchSet: 2
Gerrit-Owner: Bikramjeet Vig <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>