Marcel Kornacker has posted comments on this change.

Change subject: IMPALA-3084: Cache the sequence of table ref and materialized 
tuple ids during analysis.
......................................................................


Patch Set 7:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2367/7/testdata/workloads/functional-planner/queries/PlannerTest/tpch-nested.test
File testdata/workloads/functional-planner/queries/PlannerTest/tpch-nested.test:

Line 1922: # Q18 - Large Value Customer Query
shouldn't this really be

select c_name, c_custkey, o_orderkey, o_orderdate, o_totalprice, l.sum_qty

from customer c, c.c_orders o,

(select sum(l_quantity) as sum_qty from o.o_lineitem l having sum(l_quantity) > 
300) l

order by o_totalprice desc, o_orderdate
?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I298b8695c9f26644a395ca9f0e86040e3f5f3846
Gerrit-PatchSet: 7
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.5.0_5.7.0
Gerrit-Owner: Alex Behm <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Marcel Kornacker <[email protected]>
Gerrit-Reviewer: Silvius Rus <[email protected]>
Gerrit-HasComments: Yes

Reply via email to