NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit63e045c

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 parent0177fc7 commit63e045cCopy full SHA for 63e045c

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
`@@ -1887,7 +1887,7 @@ WHERE p1.oid = a1.amhandler AND a1.amtype = 'i' AND`
`1887`	`1887`	`-- Check for table amhandler functions with the wrong signature`
`1888`	`1888`	`SELECT a1.oid, a1.amname, p1.oid, p1.proname`
`1889`	`1889`	`FROM pg_am AS a1, pg_proc AS p1`
`1890`		`-WHERE p1.oid = a1.amhandler AND a1.amtype = 's' AND`
	`1890`	`+WHERE p1.oid = a1.amhandler AND a1.amtype = 't' AND`
`1891`	`1891`	`(p1.prorettype != 'table_am_handler'::regtype`
`1892`	`1892`	`OR p1.proretset`
`1893`	`1893`	`OR p1.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 p1.oid = a1.amhandler AND a1.amtype = 'i' AND`
`1195`	`1195`
`1196`	`1196`	`SELECT a1.oid, a1.amname, p1.oid, p1.proname`
`1197`	`1197`	`FROM pg_am AS a1, pg_proc AS p1`
`1198`		`-WHERE p1.oid = a1.amhandler AND a1.amtype = 's' AND`
	`1198`	`+WHERE p1.oid = a1.amhandler AND a1.amtype = 't' AND`
`1199`	`1199`	`(p1.prorettype != 'table_am_handler'::regtype`
`1200`	`1200`	`OR p1.proretset`
`1201`	`1201`	`OR p1.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!

Commit63e045c

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