NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit9eb80f2

committed

Don't try to use a unopened relation

Commit4c9d090 mistakenly introduced a call toTransferPredicateLocksToHeapRelation() on an index relation that hadbeen closed a few lines above. Moving up an index_open() call that'sbelow is enough to fix the problem.Discovered by me while testing an unrelated patch.

1 parent663c68f commit9eb80f2Copy full SHA for 9eb80f2

File tree

1 file changed

-3

lines changed

src/backend/catalog
- index.c

1 file changed

-3

lines changed

`‎src/backend/catalog/index.c`

Lines changed: 2 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -1464,6 +1464,8 @@ index_drop(Oid indexId, bool concurrent)`
`1464`	`1464`	`* conflicts with existing predicate locks, so now is the time to move`
`1465`	`1465`	`* them to the heap relation.`
`1466`	`1466`	`*/`
	`1467`	`+userHeapRelation=heap_open(heapId,ShareUpdateExclusiveLock);`
	`1468`	`+userIndexRelation=index_open(indexId,ShareUpdateExclusiveLock);`
`1467`	`1469`	`TransferPredicateLocksToHeapRelation(userIndexRelation);`
`1468`	`1470`
`1469`	`1471`	`/*`
`@@ -1473,9 +1475,6 @@ index_drop(Oid indexId, bool concurrent)`
`1473`	`1475`	`*/`
`1474`	`1476`	`indexRelation=heap_open(IndexRelationId,RowExclusiveLock);`
`1475`	`1477`
`1476`		`-userHeapRelation=heap_open(heapId,ShareUpdateExclusiveLock);`
`1477`		`-userIndexRelation=index_open(indexId,ShareUpdateExclusiveLock);`
`1478`		`-`
`1479`	`1478`	`tuple=SearchSysCacheCopy1(INDEXRELID,`
`1480`	`1479`	`ObjectIdGetDatum(indexId));`
`1481`	`1480`	`if (!HeapTupleIsValid(tuple))`

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!

Commit9eb80f2

File tree

1 file changed

1 file changed

`‎src/backend/catalog/index.c`

0 commit comments