Thanks to visit codestin.com
Credit goes to github.com

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

+22-4
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

+22-4
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

+22-4
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

+22-4
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

+22-4
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

+22-5
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

+22-5
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

+22-5
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

+22-5
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

+22-5
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;

sql/pathman_only.sql

+22-4
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

1533
\set VERBOSITY terse

0 commit comments

Comments
 (0)