Skip to content

Commit

Permalink
Enable analyzer test
Browse files Browse the repository at this point in the history
The underlying issue was fixed some time ago.
  • Loading branch information
martint authored and findepi committed Jul 22, 2019
1 parent c6c0859 commit 1628227
Showing 1 changed file with 5 additions and 6 deletions.
Expand Up @@ -1379,12 +1379,11 @@ public void testLambdaWithSubquery()
"line 1:34: Subquery uses 'a' which must appear in GROUP BY clause",
"SELECT (SELECT apply(0, x -> x + a) FROM (VALUES 1) x(c)) " +
"FROM t1 u GROUP BY b");
// TODO https://github.com/prestodb/presto/issues/7784
// assertFails(
// MUST_BE_AGGREGATE_OR_GROUP_BY,
// "line 1:34: Subquery uses '\"u.a\"' which must appear in GROUP BY clause",
// "SELECT (SELECT apply(0, x -> x + u.a) from (values 1) x(a)) " +
// "FROM t1 u GROUP BY b");
assertFails(
MUST_BE_AGGREGATE_OR_GROUP_BY,
"line 1:34: Subquery uses 'u.a' which must appear in GROUP BY clause",
"SELECT (SELECT apply(0, x -> x + u.a) from (values 1) x(a)) " +
"FROM t1 u GROUP BY b");

// name shadowing
analyze("SELECT (SELECT apply(0, x -> x + a) FROM (VALUES 1) x(a)) FROM t1 u GROUP BY b");
Expand Down

0 comments on commit 1628227

Please sign in to comment.