NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit06b9f38

Etsuro Fujita

committed

Clean up some misplaced comments in partition_join.sql regression test.

Also, add a comment explaining a test case.Back-patch to 11 where the regression test was added.Discussion:https://postgr.es/m/CAPmGK15adZPh2B%2BmGUjSOMH%2BH39ogDRWfCfm4G6jncZCAs9V_Q%40mail.gmail.com

1 parent332584d commit06b9f38Copy full SHA for 06b9f38

File tree

2 files changed

+13

-8

lines changed

src/test/regress
- expected
  - partition_join.out
- sql
  - partition_join.sql

2 files changed

+13

-8

lines changed

`‎src/test/regress/expected/partition_join.out‎`

Lines changed: 6 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -1857,8 +1857,8 @@ SELECT t1.a, t1.c, t2.b, t2.c FROM prt1_m t1 LEFT JOIN prt2_m t2 ON t1.c = t2.c;`
`1857`	`1857`	`-> Seq Scan on prt2_m_p3 t2_2`
`1858`	`1858`	`(11 rows)`
`1859`	`1859`
`1860`		`--- partitionwise join can not be applied betweentables with different`
`1861`		`---partition lists`
	`1860`	`+-- partitionwise join can not be appliedfor a joinbetweenlist and range`
	`1861`	`+--partitioned tables`
`1862`	`1862`	`EXPLAIN (COSTS OFF)`
`1863`	`1863`	`SELECT t1.a, t1.c, t2.b, t2.c FROM prt1_n t1 LEFT JOIN prt2_n t2 ON (t1.c = t2.c);`
`1864`	`1864`	`QUERY PLAN`
`@@ -1874,6 +1874,8 @@ SELECT t1.a, t1.c, t2.b, t2.c FROM prt1_n t1 LEFT JOIN prt2_n t2 ON (t1.c = t2.c`
`1874`	`1874`	`-> Seq Scan on prt1_n_p2 t1_1`
`1875`	`1875`	`(9 rows)`
`1876`	`1876`
	`1877`	`+-- partitionwise join can not be applied between tables with different`
	`1878`	`+-- partition lists`
`1877`	`1879`	`EXPLAIN (COSTS OFF)`
`1878`	`1880`	`SELECT t1.a, t1.c, t2.b, t2.c FROM prt1_n t1 JOIN prt2_n t2 ON (t1.c = t2.c) JOIN plt1 t3 ON (t1.c = t3.c);`
`1879`	`1881`	`QUERY PLAN`
`@@ -1896,8 +1898,8 @@ SELECT t1.a, t1.c, t2.b, t2.c FROM prt1_n t1 JOIN prt2_n t2 ON (t1.c = t2.c) JOI`
`1896`	`1898`	`-> Seq Scan on prt1_n_p2 t1_1`
`1897`	`1899`	`(16 rows)`
`1898`	`1900`
`1899`		`--- partitionwise join can not be applied for a join betweenlist and range`
`1900`		`---partitioned tables`
	`1901`	`+-- partitionwise join can not be applied for a join betweenkey column and`
	`1902`	`+--non-key column`
`1901`	`1903`	`EXPLAIN (COSTS OFF)`
`1902`	`1904`	`SELECT t1.a, t1.c, t2.b, t2.c FROM prt1_n t1 FULL JOIN prt1 t2 ON (t1.c = t2.c);`
`1903`	`1905`	`QUERY PLAN`

`‎src/test/regress/sql/partition_join.sql‎`

Lines changed: 7 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -394,14 +394,17 @@ SELECT t1.a, t1.c, t2.b, t2.c FROM prt1_m t1 LEFT JOIN prt2_m t2 ON t1.a = t2.b;`
`394`	`394`	`EXPLAIN (COSTS OFF)`
`395`	`395`	`SELECTt1.a,t1.c,t2.b,t2.cFROM prt1_m t1LEFT JOIN prt2_m t2ONt1.c=t2.c;`
`396`	`396`
`397`		`--- partitionwise join can not be applied betweentables with different`
`398`		`---partition lists`
	`397`	`+-- partitionwise join can not be appliedfor a joinbetweenlist and range`
	`398`	`+--partitioned tables`
`399`	`399`	`EXPLAIN (COSTS OFF)`
`400`	`400`	`SELECTt1.a,t1.c,t2.b,t2.cFROM prt1_n t1LEFT JOIN prt2_n t2ON (t1.c=t2.c);`
	`401`	`+`
	`402`	`+-- partitionwise join can not be applied between tables with different`
	`403`	`+-- partition lists`
`401`	`404`	`EXPLAIN (COSTS OFF)`
`402`	`405`	`SELECTt1.a,t1.c,t2.b,t2.cFROM prt1_n t1JOIN prt2_n t2ON (t1.c=t2.c)JOIN plt1 t3ON (t1.c=t3.c);`
`403`	`406`
`404`		`--- partitionwise join can not be applied for a join betweenlist and range`
`405`		`---partitioned tables`
	`407`	`+-- partitionwise join can not be applied for a join betweenkey column and`
	`408`	`+--non-key column`
`406`	`409`	`EXPLAIN (COSTS OFF)`
`407`	`410`	`SELECTt1.a,t1.c,t2.b,t2.cFROM prt1_n t1 FULLJOIN prt1 t2ON (t1.c=t2.c);`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit06b9f38

File tree

2 files changed

2 files changed

`‎src/test/regress/expected/partition_join.out‎`

`‎src/test/regress/sql/partition_join.sql‎`

0 commit comments