Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15285 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/15285 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- 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(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- 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: merged Gerrit-Change-Id: I0610b93b9a13da19e35151b231349c0ab16071d2 Gerrit-Change-Number: 15285 Gerrit-PatchSet: 4 Gerrit-Owner: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
