Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/16052 )
Change subject: IMPALA-9784, IMPALA-9905: Uncorrelated subqueries in HAVING. ...................................................................... Patch Set 3: Code-Review+1 (1 comment) Assume Aman probably wants to have a look, but LGTM once tests pass. http://gerrit.cloudera.org:8080/#/c/16052/3/testdata/workloads/functional-planner/queries/PlannerTest/subquery-rewrite.test File testdata/workloads/functional-planner/queries/PlannerTest/subquery-rewrite.test: http://gerrit.cloudera.org:8080/#/c/16052/3/testdata/workloads/functional-planner/queries/PlannerTest/subquery-rewrite.test@3300 PS3, Line 3300: returns nit: return -- To view, visit http://gerrit.cloudera.org:8080/16052 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I124a58a09a1a47e1222a22d84b54fe7d07844461 Gerrit-Change-Number: 16052 Gerrit-PatchSet: 3 Gerrit-Owner: Shant Hovsepian <sh...@cloudera.com> Gerrit-Reviewer: Aman Sinha <amsi...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Shant Hovsepian <sh...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Comment-Date: Mon, 29 Jun 2020 17:13:56 +0000 Gerrit-HasComments: Yes