NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit3dadeef

committed

Fix query checking consistency of table amhandlers in opr_sanity.sql

As written, the query checked for an access method of type 's', which isnot an AM type supported in the core code.Error introduced by8586bf7. As this query is not checking what itshould, backpatch all the way down.Reviewed-by: Aleksander AlekseevDiscussion:https://postgr.es/m/ZVxJkAJrKbfHETiy@paquier.xyzBackpatch-through: 12

1 parent9dd50e9 commit3dadeefCopy full SHA for 3dadeef

File tree

2 files changed

-2

lines changed

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

2 files changed

-2

lines changed

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

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1886,7 +1886,7 @@ WHERE p2.oid = p1.amhandler AND p1.amtype = 'i' AND`
`1886`	`1886`	`-- Check for table amhandler functions with the wrong signature`
`1887`	`1887`	`SELECT p1.oid, p1.amname, p2.oid, p2.proname`
`1888`	`1888`	`FROM pg_am AS p1, pg_proc AS p2`
`1889`		`-WHERE p2.oid = p1.amhandler AND p1.amtype = 's' AND`
	`1889`	`+WHERE p2.oid = p1.amhandler AND p1.amtype = 't' AND`
`1890`	`1890`	`(p2.prorettype != 'table_am_handler'::regtype`
`1891`	`1891`	`OR p2.proretset`
`1892`	`1892`	`OR p2.pronargs != 1`

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

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1195,7 +1195,7 @@ WHERE p2.oid = p1.amhandler AND p1.amtype = 'i' AND`
`1195`	`1195`
`1196`	`1196`	`SELECT p1.oid, p1.amname, p2.oid, p2.proname`
`1197`	`1197`	`FROM pg_am AS p1, pg_proc AS p2`
`1198`		`-WHERE p2.oid = p1.amhandler AND p1.amtype = 's' AND`
	`1198`	`+WHERE p2.oid = p1.amhandler AND p1.amtype = 't' AND`
`1199`	`1199`	`(p2.prorettype != 'table_am_handler'::regtype`
`1200`	`1200`	`OR p2.proretset`
`1201`	`1201`	`OR p2.pronargs != 1`

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!

Commit3dadeef

File tree

2 files changed

2 files changed

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

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

0 commit comments