[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-02-07 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. Patch Set 5: (56 comments) Thanks for the reviews. http://gerrit.cloudera.org:8080/#/c/5816/5//COMMIT

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-02-01 Thread Alex Behm (Code Review)
Alex Behm has posted comments on this change. Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. Patch Set 5: (42 comments) http://gerrit.cloudera.org:8080/#/c/5816/5//COMMIT_MSG Commit Message: Line 14: Te

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-02-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5816/4/testdata/workloads/functional-query/qu

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-02-01 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. Patch Set 4: (15 comments) I think this is looking pretty good. It turns out to hit a lot of interesting

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-02-01 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#5). Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. IMPALA-3586 (Part 1): Implement Union Pass Through The union node acts as pass through operator and forwa

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-02-01 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#5). Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. IMPALA-3586 (Part 1): Implement Union Pass Through The union node acts as pass through operator and forwa

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-01-31 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#4). Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. IMPALA-3586 (Part 1): Implement Union Pass Through The union node acts as pass through operator and forwa

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-01-31 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#4). Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. IMPALA-3586 (Part 1): Implement Union Pass Through The union node acts as pass through operator and forwa

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-01-31 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#4). Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. IMPALA-3586 (Part 1): Implement Union Pass Through The union node acts as pass through operator and forwa

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-01-31 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#3). Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. IMPALA-3586 (Part 1): Implement Union Pass Through The union node acts as pass through operator and forwa

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-01-31 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has posted comments on this change. Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. Patch Set 3: (6 comments) http://gerrit.cloudera.org:8080/#/c/5816/2/be/src/exec/union-node.cc File be

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-01-31 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#3). Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. IMPALA-3586 (Part 1): Implement Union Pass Through The union node acts as pass through operator and forwa

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-01-27 Thread Tim Armstrong (Code Review)
Tim Armstrong has posted comments on this change. Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. Patch Set 2: (6 comments) Need to look at fe and tests, but had some comments on the backend. http://gerr

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-01-27 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new patch set (#2). Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. IMPALA-3586 (Part 1): Implement Union Pass Through The union node acts as pass through operator and forwa

[Impala-ASF-CR] IMPALA-3586 (Part 1): Implement Union Pass Through

2017-01-27 Thread Taras Bobrovytsky (Code Review)
Taras Bobrovytsky has uploaded a new change for review. http://gerrit.cloudera.org:8080/5816 Change subject: IMPALA-3586 (Part 1): Implement Union Pass Through .. IMPALA-3586 (Part 1): Implement Union Pass Through The union n