Marcel Kornacker has posted comments on this change.

Change subject: Improve the SQL for nested TPCH-Q18.
......................................................................


Patch Set 1: Code-Review+2

(1 comment)

hm, this is going to mess with our baseline. we should have a process for 
incorporating workload changes.

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

Line 1958: |  |  |  having: sum(l_quantity) > 300
nice, ends up as a having clause


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8b351b7f467e8bef0c256dc43cea325d7f177edf
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Alex Behm <[email protected]>
Gerrit-Reviewer: Marcel Kornacker <[email protected]>
Gerrit-HasComments: Yes

Reply via email to