aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRobert Haas <rhaas@postgresql.org>2016-10-21 11:27:32 -0400
committerRobert Haas <rhaas@postgresql.org>2016-10-21 11:29:33 -0400
commitad13a09d762f0c903a52ed0ec668a0ba51a61047 (patch)
treebbab132923f6edd55cd0eb379f817f627d84fcec
parent7aa2c10ac6785a2de683609b98da607e588a6d02 (diff)
downloadpostgresql-ad13a09d762f0c903a52ed0ec668a0ba51a61047.tar.gz
postgresql-ad13a09d762f0c903a52ed0ec668a0ba51a61047.zip
postgres_fdw: Attempt to stabilize regression results.
Set enable_hashagg to false for tests involving least_agg(), so that we get the same plan regardless of local costing variances. Also, remove a test involving sqrt(); it's there to test deparsing of HAVING clauses containing expressions, but that's tested elsewhere anyway, and sqrt(2) deparses with different amounts of precision on different machines. Per buildfarm.
-rw-r--r--contrib/postgres_fdw/expected/postgres_fdw.out74
-rw-r--r--contrib/postgres_fdw/sql/postgres_fdw.sql9
2 files changed, 25 insertions, 58 deletions
diff --git a/contrib/postgres_fdw/expected/postgres_fdw.out b/contrib/postgres_fdw/expected/postgres_fdw.out
index d3f37adad6c..88b696cedeb 100644
--- a/contrib/postgres_fdw/expected/postgres_fdw.out
+++ b/contrib/postgres_fdw/expected/postgres_fdw.out
@@ -2491,35 +2491,6 @@ select c2, sum(c1) from ft2 group by c2 having avg(c1) < 500 and sum(c1) < 49800
2 | 49700
(2 rows)
--- Using expressions in HAVING clause
-explain (verbose, costs off)
-select c5, count(c2) from ft1 group by c5, sqrt(c2) having sqrt(max(c2)) = sqrt(2) order by 1, 2;
- QUERY PLAN
---------------------------------------------------------------------------------------------------------------------------------------------------------------
- Sort
- Output: c5, (count(c2)), (sqrt((c2)::double precision))
- Sort Key: ft1.c5, (count(ft1.c2))
- -> Foreign Scan
- Output: c5, (count(c2)), (sqrt((c2)::double precision))
- Relations: Aggregate on (public.ft1)
- Remote SQL: SELECT c5, count(c2), sqrt(c2) FROM "S 1"."T 1" GROUP BY c5, (sqrt(c2)) HAVING ((sqrt(max(c2)) = 1.41421356237309515::double precision))
-(7 rows)
-
-select c5, count(c2) from ft1 group by c5, sqrt(c2) having sqrt(max(c2)) = sqrt(2) order by 1, 2;
- c5 | count
---------------------------+-------
- Sat Jan 03 00:00:00 1970 | 10
- Tue Jan 13 00:00:00 1970 | 10
- Fri Jan 23 00:00:00 1970 | 10
- Mon Feb 02 00:00:00 1970 | 10
- Thu Feb 12 00:00:00 1970 | 10
- Sun Feb 22 00:00:00 1970 | 10
- Wed Mar 04 00:00:00 1970 | 10
- Sat Mar 14 00:00:00 1970 | 10
- Tue Mar 24 00:00:00 1970 | 10
- Fri Apr 03 00:00:00 1970 | 10
-(10 rows)
-
-- Unshippable HAVING clause will be evaluated locally, and other qual in HAVING clause is pushed down
explain (verbose, costs off)
select count(*) from (select c5, count(c1) from ft1 group by c5, sqrt(c2) having (avg(c1) / avg(c1)) * random() <= 1 and avg(c1) < 500) x;
@@ -2844,21 +2815,20 @@ returns anyelement language sql as
create aggregate least_agg(variadic items anyarray) (
stype = anyelement, sfunc = least_accum
);
+-- Disable hash aggregation for plan stability.
+set enable_hashagg to false;
-- Not pushed down due to user defined aggregate
explain (verbose, costs off)
select c2, least_agg(c1) from ft1 group by c2 order by c2;
- QUERY PLAN
--------------------------------------------------------------
- Sort
- Output: c2, (least_agg(VARIADIC ARRAY[c1]))
- Sort Key: ft1.c2
- -> HashAggregate
- Output: c2, least_agg(VARIADIC ARRAY[c1])
- Group Key: ft1.c2
- -> Foreign Scan on public.ft1
- Output: c2, c1
- Remote SQL: SELECT "C 1", c2 FROM "S 1"."T 1"
-(9 rows)
+ QUERY PLAN
+----------------------------------------------------------------------------------
+ GroupAggregate
+ Output: c2, least_agg(VARIADIC ARRAY[c1])
+ Group Key: ft1.c2
+ -> Foreign Scan on public.ft1
+ Output: c2, c1
+ Remote SQL: SELECT "C 1", c2 FROM "S 1"."T 1" ORDER BY c2 ASC NULLS LAST
+(6 rows)
-- Add function and aggregate into extension
alter extension postgres_fdw add function least_accum(anyelement, variadic anyarray);
@@ -2900,20 +2870,18 @@ alter server loopback options (set extensions 'postgres_fdw');
-- Not pushed down as we have dropped objects from extension.
explain (verbose, costs off)
select c2, least_agg(c1) from ft1 group by c2 order by c2;
- QUERY PLAN
--------------------------------------------------------------
- Sort
- Output: c2, (least_agg(VARIADIC ARRAY[c1]))
- Sort Key: ft1.c2
- -> HashAggregate
- Output: c2, least_agg(VARIADIC ARRAY[c1])
- Group Key: ft1.c2
- -> Foreign Scan on public.ft1
- Output: c2, c1
- Remote SQL: SELECT "C 1", c2 FROM "S 1"."T 1"
-(9 rows)
+ QUERY PLAN
+----------------------------------------------------------------------------------
+ GroupAggregate
+ Output: c2, least_agg(VARIADIC ARRAY[c1])
+ Group Key: ft1.c2
+ -> Foreign Scan on public.ft1
+ Output: c2, c1
+ Remote SQL: SELECT "C 1", c2 FROM "S 1"."T 1" ORDER BY c2 ASC NULLS LAST
+(6 rows)
-- Cleanup
+reset enable_hashagg;
drop aggregate least_agg(variadic items anyarray);
drop function least_accum(anyelement, variadic anyarray);
-- Testing USING OPERATOR() in ORDER BY within aggregate.
diff --git a/contrib/postgres_fdw/sql/postgres_fdw.sql b/contrib/postgres_fdw/sql/postgres_fdw.sql
index aeed8f62de4..bb9d41a1b33 100644
--- a/contrib/postgres_fdw/sql/postgres_fdw.sql
+++ b/contrib/postgres_fdw/sql/postgres_fdw.sql
@@ -593,11 +593,6 @@ explain (verbose, costs off)
select c2, sum(c1) from ft2 group by c2 having avg(c1) < 500 and sum(c1) < 49800 order by c2;
select c2, sum(c1) from ft2 group by c2 having avg(c1) < 500 and sum(c1) < 49800 order by c2;
--- Using expressions in HAVING clause
-explain (verbose, costs off)
-select c5, count(c2) from ft1 group by c5, sqrt(c2) having sqrt(max(c2)) = sqrt(2) order by 1, 2;
-select c5, count(c2) from ft1 group by c5, sqrt(c2) having sqrt(max(c2)) = sqrt(2) order by 1, 2;
-
-- Unshippable HAVING clause will be evaluated locally, and other qual in HAVING clause is pushed down
explain (verbose, costs off)
select count(*) from (select c5, count(c1) from ft1 group by c5, sqrt(c2) having (avg(c1) / avg(c1)) * random() <= 1 and avg(c1) < 500) x;
@@ -677,6 +672,9 @@ create aggregate least_agg(variadic items anyarray) (
stype = anyelement, sfunc = least_accum
);
+-- Disable hash aggregation for plan stability.
+set enable_hashagg to false;
+
-- Not pushed down due to user defined aggregate
explain (verbose, costs off)
select c2, least_agg(c1) from ft1 group by c2 order by c2;
@@ -701,6 +699,7 @@ explain (verbose, costs off)
select c2, least_agg(c1) from ft1 group by c2 order by c2;
-- Cleanup
+reset enable_hashagg;
drop aggregate least_agg(variadic items anyarray);
drop function least_accum(anyelement, variadic anyarray);