Skip to content

Commit d41acb6

Browse files
author
Ekaterina Sokolova
committed
Fix comments in test files due to alternate outputs.
1 parent 650cef6 commit d41acb6

12 files changed

+264
-54
lines changed

expected/pathman_only.out

Lines changed: 22 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,31 @@
33
* NOTE: This test behaves differenly on PgPro
44
* ---------------------------------------------
55
*
6-
* Since 12 (608b167f9f), CTEs which are scanned once are no longer an
7-
* optimization fence, which changes practically all plans here. There is
6+
* --------------------
7+
* pathman_only_1.sql
8+
* --------------------
9+
* Since 608b167f9f in PostgreSQL 12, CTEs which are scanned once are no longer
10+
* an optimization fence, which changes practically all plans here. There is
811
* an option to forcibly make them MATERIALIZED, but we also need to run tests
912
* on older versions, so create pathman_only_1.out instead.
1013
*
11-
* Since 55a1954da16 and 6ef77cf46e8 (>= 13) output of EXPLAIN was changed,
12-
* now it includes aliases for inherited tables.
14+
* --------------------
15+
* pathman_only_2.sql
16+
* --------------------
17+
* Since 55a1954da16 and 6ef77cf46e8 in PostgreSQL 13, output of EXPLAIN was
18+
* changed, now it includes aliases for inherited tables.
19+
*
20+
* --------------------
21+
* pathman_only_3.sql
22+
* --------------------
23+
* Since a5fc46414de in PostgreSQL 16, the order of the operands was changed,
24+
* which affected the output of the "Prune by" in EXPLAIN.
25+
*
26+
* --------------------
27+
* pathman_only_4.sql
28+
* --------------------
29+
* Since fd0398fcb09 in PostgreSQL 17, output of EXPLAIN was
30+
* changed, now it displays SubPlan nodes and output parameters.
1331
*/
1432
\set VERBOSITY terse
1533
SET search_path = 'public';

expected/pathman_only_1.out

Lines changed: 22 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,31 @@
33
* NOTE: This test behaves differenly on PgPro
44
* ---------------------------------------------
55
*
6-
* Since 12 (608b167f9f), CTEs which are scanned once are no longer an
7-
* optimization fence, which changes practically all plans here. There is
6+
* --------------------
7+
* pathman_only_1.sql
8+
* --------------------
9+
* Since 608b167f9f in PostgreSQL 12, CTEs which are scanned once are no longer
10+
* an optimization fence, which changes practically all plans here. There is
811
* an option to forcibly make them MATERIALIZED, but we also need to run tests
912
* on older versions, so create pathman_only_1.out instead.
1013
*
11-
* Since 55a1954da16 and 6ef77cf46e8 (>= 13) output of EXPLAIN was changed,
12-
* now it includes aliases for inherited tables.
14+
* --------------------
15+
* pathman_only_2.sql
16+
* --------------------
17+
* Since 55a1954da16 and 6ef77cf46e8 in PostgreSQL 13, output of EXPLAIN was
18+
* changed, now it includes aliases for inherited tables.
19+
*
20+
* --------------------
21+
* pathman_only_3.sql
22+
* --------------------
23+
* Since a5fc46414de in PostgreSQL 16, the order of the operands was changed,
24+
* which affected the output of the "Prune by" in EXPLAIN.
25+
*
26+
* --------------------
27+
* pathman_only_4.sql
28+
* --------------------
29+
* Since fd0398fcb09 in PostgreSQL 17, output of EXPLAIN was
30+
* changed, now it displays SubPlan nodes and output parameters.
1331
*/
1432
\set VERBOSITY terse
1533
SET search_path = 'public';

expected/pathman_only_2.out

Lines changed: 22 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,31 @@
33
* NOTE: This test behaves differenly on PgPro
44
* ---------------------------------------------
55
*
6-
* Since 12 (608b167f9f), CTEs which are scanned once are no longer an
7-
* optimization fence, which changes practically all plans here. There is
6+
* --------------------
7+
* pathman_only_1.sql
8+
* --------------------
9+
* Since 608b167f9f in PostgreSQL 12, CTEs which are scanned once are no longer
10+
* an optimization fence, which changes practically all plans here. There is
811
* an option to forcibly make them MATERIALIZED, but we also need to run tests
912
* on older versions, so create pathman_only_1.out instead.
1013
*
11-
* Since 55a1954da16 and 6ef77cf46e8 (>= 13) output of EXPLAIN was changed,
12-
* now it includes aliases for inherited tables.
14+
* --------------------
15+
* pathman_only_2.sql
16+
* --------------------
17+
* Since 55a1954da16 and 6ef77cf46e8 in PostgreSQL 13, output of EXPLAIN was
18+
* changed, now it includes aliases for inherited tables.
19+
*
20+
* --------------------
21+
* pathman_only_3.sql
22+
* --------------------
23+
* Since a5fc46414de in PostgreSQL 16, the order of the operands was changed,
24+
* which affected the output of the "Prune by" in EXPLAIN.
25+
*
26+
* --------------------
27+
* pathman_only_4.sql
28+
* --------------------
29+
* Since fd0398fcb09 in PostgreSQL 17, output of EXPLAIN was
30+
* changed, now it displays SubPlan nodes and output parameters.
1331
*/
1432
\set VERBOSITY terse
1533
SET search_path = 'public';

expected/pathman_only_3.out

Lines changed: 22 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,31 @@
33
* NOTE: This test behaves differenly on PgPro
44
* ---------------------------------------------
55
*
6-
* Since 12 (608b167f9f), CTEs which are scanned once are no longer an
7-
* optimization fence, which changes practically all plans here. There is
6+
* --------------------
7+
* pathman_only_1.sql
8+
* --------------------
9+
* Since 608b167f9f in PostgreSQL 12, CTEs which are scanned once are no longer
10+
* an optimization fence, which changes practically all plans here. There is
811
* an option to forcibly make them MATERIALIZED, but we also need to run tests
912
* on older versions, so create pathman_only_1.out instead.
1013
*
11-
* Since 55a1954da16 and 6ef77cf46e8 (>= 13) output of EXPLAIN was changed,
12-
* now it includes aliases for inherited tables.
14+
* --------------------
15+
* pathman_only_2.sql
16+
* --------------------
17+
* Since 55a1954da16 and 6ef77cf46e8 in PostgreSQL 13, output of EXPLAIN was
18+
* changed, now it includes aliases for inherited tables.
19+
*
20+
* --------------------
21+
* pathman_only_3.sql
22+
* --------------------
23+
* Since a5fc46414de in PostgreSQL 16, the order of the operands was changed,
24+
* which affected the output of the "Prune by" in EXPLAIN.
25+
*
26+
* --------------------
27+
* pathman_only_4.sql
28+
* --------------------
29+
* Since fd0398fcb09 in PostgreSQL 17, output of EXPLAIN was
30+
* changed, now it displays SubPlan nodes and output parameters.
1331
*/
1432
\set VERBOSITY terse
1533
SET search_path = 'public';

expected/pathman_only_4.out

Lines changed: 22 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,31 @@
33
* NOTE: This test behaves differenly on PgPro
44
* ---------------------------------------------
55
*
6-
* Since 12 (608b167f9f), CTEs which are scanned once are no longer an
7-
* optimization fence, which changes practically all plans here. There is
6+
* --------------------
7+
* pathman_only_1.sql
8+
* --------------------
9+
* Since 608b167f9f in PostgreSQL 12, CTEs which are scanned once are no longer
10+
* an optimization fence, which changes practically all plans here. There is
811
* an option to forcibly make them MATERIALIZED, but we also need to run tests
912
* on older versions, so create pathman_only_1.out instead.
1013
*
11-
* Since 55a1954da16 and 6ef77cf46e8 (>= 13) output of EXPLAIN was changed,
12-
* now it includes aliases for inherited tables.
14+
* --------------------
15+
* pathman_only_2.sql
16+
* --------------------
17+
* Since 55a1954da16 and 6ef77cf46e8 in PostgreSQL 13, output of EXPLAIN was
18+
* changed, now it includes aliases for inherited tables.
19+
*
20+
* --------------------
21+
* pathman_only_3.sql
22+
* --------------------
23+
* Since a5fc46414de in PostgreSQL 16, the order of the operands was changed,
24+
* which affected the output of the "Prune by" in EXPLAIN.
25+
*
26+
* --------------------
27+
* pathman_only_4.sql
28+
* --------------------
29+
* Since fd0398fcb09 in PostgreSQL 17, output of EXPLAIN was
30+
* changed, now it displays SubPlan nodes and output parameters.
1331
*/
1432
\set VERBOSITY terse
1533
SET search_path = 'public';

expected/pathman_rowmarks.out

Lines changed: 22 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,30 @@
11
/*
22
* -------------------------------------------
3-
* NOTE: This test behaves differenly on 9.5
3+
* NOTE: This test behaves differenly on PgPro
44
* -------------------------------------------
55
*
6-
* Also since 8edd0e794 (>= 12) Append nodes with single subplan are eliminated,
7-
* causing different output; pathman_rowmarks_2.out is the updated version.
6+
* ------------------------
7+
* pathman_rowmarks_1.sql
8+
* ------------------------
9+
* Since PostgreSQL 9.5, output of EXPLAIN was changed.
810
*
9-
* Since 55a1954da16 and 6ef77cf46e8 (>= 13) output of EXPLAIN was changed,
10-
* now it includes aliases for inherited tables.
11+
* ------------------------
12+
* pathman_rowmarks_2.sql
13+
* ------------------------
14+
* Since 8edd0e794 in PostgreSQL 12, append nodes with single subplan are
15+
* eliminated, causing different output.
16+
*
17+
* ------------------------
18+
* pathman_rowmarks_3.sql
19+
* ------------------------
20+
* Since 55a1954da16 and 6ef77cf46e8 in PostgreSQL 13 output of EXPLAIN was
21+
* changed, now it includes aliases for inherited tables.
22+
*
23+
* ------------------------
24+
* pathman_rowmarks_3.sql
25+
* ------------------------
26+
* Since fd0398fcb09 in PostgreSQL 17, output of EXPLAIN was
27+
* changed, now it displays SubPlan nodes and output parameters.
1128
*/
1229
SET search_path = 'public';
1330
CREATE EXTENSION pg_pathman;

expected/pathman_rowmarks_1.out

Lines changed: 22 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,30 @@
11
/*
22
* -------------------------------------------
3-
* NOTE: This test behaves differenly on 9.5
3+
* NOTE: This test behaves differenly on PgPro
44
* -------------------------------------------
55
*
6-
* Also since 8edd0e794 (>= 12) Append nodes with single subplan are eliminated,
7-
* causing different output; pathman_rowmarks_2.out is the updated version.
6+
* ------------------------
7+
* pathman_rowmarks_1.sql
8+
* ------------------------
9+
* Since PostgreSQL 9.5, output of EXPLAIN was changed.
810
*
9-
* Since 55a1954da16 and 6ef77cf46e8 (>= 13) output of EXPLAIN was changed,
10-
* now it includes aliases for inherited tables.
11+
* ------------------------
12+
* pathman_rowmarks_2.sql
13+
* ------------------------
14+
* Since 8edd0e794 in PostgreSQL 12, append nodes with single subplan are
15+
* eliminated, causing different output.
16+
*
17+
* ------------------------
18+
* pathman_rowmarks_3.sql
19+
* ------------------------
20+
* Since 55a1954da16 and 6ef77cf46e8 in PostgreSQL 13 output of EXPLAIN was
21+
* changed, now it includes aliases for inherited tables.
22+
*
23+
* ------------------------
24+
* pathman_rowmarks_3.sql
25+
* ------------------------
26+
* Since fd0398fcb09 in PostgreSQL 17, output of EXPLAIN was
27+
* changed, now it displays SubPlan nodes and output parameters.
1128
*/
1229
SET search_path = 'public';
1330
CREATE EXTENSION pg_pathman;

expected/pathman_rowmarks_2.out

Lines changed: 22 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,30 @@
11
/*
22
* -------------------------------------------
3-
* NOTE: This test behaves differenly on 9.5
3+
* NOTE: This test behaves differenly on PgPro
44
* -------------------------------------------
55
*
6-
* Also since 8edd0e794 (>= 12) Append nodes with single subplan are eliminated,
7-
* causing different output; pathman_rowmarks_2.out is the updated version.
6+
* ------------------------
7+
* pathman_rowmarks_1.sql
8+
* ------------------------
9+
* Since PostgreSQL 9.5, output of EXPLAIN was changed.
810
*
9-
* Since 55a1954da16 and 6ef77cf46e8 (>= 13) output of EXPLAIN was changed,
10-
* now it includes aliases for inherited tables.
11+
* ------------------------
12+
* pathman_rowmarks_2.sql
13+
* ------------------------
14+
* Since 8edd0e794 in PostgreSQL 12, append nodes with single subplan are
15+
* eliminated, causing different output.
16+
*
17+
* ------------------------
18+
* pathman_rowmarks_3.sql
19+
* ------------------------
20+
* Since 55a1954da16 and 6ef77cf46e8 in PostgreSQL 13 output of EXPLAIN was
21+
* changed, now it includes aliases for inherited tables.
22+
*
23+
* ------------------------
24+
* pathman_rowmarks_3.sql
25+
* ------------------------
26+
* Since fd0398fcb09 in PostgreSQL 17, output of EXPLAIN was
27+
* changed, now it displays SubPlan nodes and output parameters.
1128
*/
1229
SET search_path = 'public';
1330
CREATE EXTENSION pg_pathman;

expected/pathman_rowmarks_3.out

Lines changed: 22 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,30 @@
11
/*
22
* -------------------------------------------
3-
* NOTE: This test behaves differenly on 9.5
3+
* NOTE: This test behaves differenly on PgPro
44
* -------------------------------------------
55
*
6-
* Also since 8edd0e794 (>= 12) Append nodes with single subplan are eliminated,
7-
* causing different output; pathman_rowmarks_2.out is the updated version.
6+
* ------------------------
7+
* pathman_rowmarks_1.sql
8+
* ------------------------
9+
* Since PostgreSQL 9.5, output of EXPLAIN was changed.
810
*
9-
* Since 55a1954da16 and 6ef77cf46e8 (>= 13) output of EXPLAIN was changed,
10-
* now it includes aliases for inherited tables.
11+
* ------------------------
12+
* pathman_rowmarks_2.sql
13+
* ------------------------
14+
* Since 8edd0e794 in PostgreSQL 12, append nodes with single subplan are
15+
* eliminated, causing different output.
16+
*
17+
* ------------------------
18+
* pathman_rowmarks_3.sql
19+
* ------------------------
20+
* Since 55a1954da16 and 6ef77cf46e8 in PostgreSQL 13 output of EXPLAIN was
21+
* changed, now it includes aliases for inherited tables.
22+
*
23+
* ------------------------
24+
* pathman_rowmarks_3.sql
25+
* ------------------------
26+
* Since fd0398fcb09 in PostgreSQL 17, output of EXPLAIN was
27+
* changed, now it displays SubPlan nodes and output parameters.
1128
*/
1229
SET search_path = 'public';
1330
CREATE EXTENSION pg_pathman;

expected/pathman_rowmarks_4.out

Lines changed: 22 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,30 @@
11
/*
22
* -------------------------------------------
3-
* NOTE: This test behaves differenly on 9.5
3+
* NOTE: This test behaves differenly on PgPro
44
* -------------------------------------------
55
*
6-
* Also since 8edd0e794 (>= 12) Append nodes with single subplan are eliminated,
7-
* causing different output; pathman_rowmarks_2.out is the updated version.
6+
* ------------------------
7+
* pathman_rowmarks_1.sql
8+
* ------------------------
9+
* Since PostgreSQL 9.5, output of EXPLAIN was changed.
810
*
9-
* Since 55a1954da16 and 6ef77cf46e8 (>= 13) output of EXPLAIN was changed,
10-
* now it includes aliases for inherited tables.
11+
* ------------------------
12+
* pathman_rowmarks_2.sql
13+
* ------------------------
14+
* Since 8edd0e794 in PostgreSQL 12, append nodes with single subplan are
15+
* eliminated, causing different output.
16+
*
17+
* ------------------------
18+
* pathman_rowmarks_3.sql
19+
* ------------------------
20+
* Since 55a1954da16 and 6ef77cf46e8 in PostgreSQL 13 output of EXPLAIN was
21+
* changed, now it includes aliases for inherited tables.
22+
*
23+
* ------------------------
24+
* pathman_rowmarks_3.sql
25+
* ------------------------
26+
* Since fd0398fcb09 in PostgreSQL 17, output of EXPLAIN was
27+
* changed, now it displays SubPlan nodes and output parameters.
1128
*/
1229
SET search_path = 'public';
1330
CREATE EXTENSION pg_pathman;

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy