Taras Bobrovytsky has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9017 )

Change subject: IMPALA-6388: Fix the Union node number of hosts estimation
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/9017/1/testdata/workloads/functional-planner/queries/PlannerTest/resource-requirements.test
File 
testdata/workloads/functional-planner/queries/PlannerTest/resource-requirements.test:

http://gerrit.cloudera.org:8080/#/c/9017/1/testdata/workloads/functional-planner/queries/PlannerTest/resource-requirements.test@3561
PS1, Line 3561: # IMPALA-6388: Verify that the order of the union operands does 
not impact the
> move these into union.test
Done


http://gerrit.cloudera.org:8080/#/c/9017/1/testdata/workloads/functional-planner/queries/PlannerTest/resource-requirements.test@3563
PS1, Line 3563: select coalesce(int_col, int_col) from functional.alltypes
> why coalesce()?
We want to force the first union operand to be not passthrough. In the other 
query we force the second operand to be not passthrough.



--
To view, visit http://gerrit.cloudera.org:8080/9017
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I51e1ecca8dbc84b2b5a72708667b2799d00279f0
Gerrit-Change-Number: 9017
Gerrit-PatchSet: 1
Gerrit-Owner: Taras Bobrovytsky <tbobrovyt...@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com>
Gerrit-Reviewer: Taras Bobrovytsky <tbobrovyt...@cloudera.com>
Gerrit-Comment-Date: Fri, 12 Jan 2018 20:49:27 +0000
Gerrit-HasComments: Yes

Reply via email to