Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

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

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitd7b16ee

Browse files
committed
Improve relcache invalidation handling of currently invisible relations.
The corner case where a relcache invalidation tried to rebuild theentry for a referenced relation but couldn't find it in the catalogwasn't correct.The code tried to RelationCacheDelete/RelationDestroyRelation theentry. That didn't work when assertions are enabled because the lattercontains an assertion ensuring the refcount is zero. It's also moregenerally a bad idea, because by virtue of being referenced somebodymight actually look at the entry, which is possible if the error istrapped and handled via a subtransaction abort.Instead just error out, without deleting the entry. As the entry ismarked invalid, the worst that can happen is that the invalid (and atsome point unused) entry lingers in the relcache.Discussion: 22459.1418656530@sss.pgh.pa.usThere should be no way to hit this case < 9.4 where logical decodingintroduced a bug that can hit this. But since the code for handlingthe corner case is there it should do something halfway sane, sobackpatch all the the way back. The logical decoding bug will behandled in a separate commit.
1 parent97f8a24 commitd7b16ee

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

‎src/backend/utils/cache/relcache.c

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1952,9 +1952,13 @@ RelationClearRelation(Relation relation, bool rebuild)
19521952
newrel=RelationBuildDesc(save_relid, false);
19531953
if (newrel==NULL)
19541954
{
1955-
/* Should only get here if relation was deleted */
1956-
RelationCacheDelete(relation);
1957-
RelationDestroyRelation(relation);
1955+
/*
1956+
* This shouldn't happen as dropping a relation is intended to be
1957+
* impossible if still referenced (c.f. CheckTableNotInUse()). But
1958+
* if we get here anyway, we can't just delete the relcache entry,
1959+
* as it possibly could get accessed later (as e.g. the error
1960+
* might get trapped and handled via a subtransaction rollback).
1961+
*/
19581962
elog(ERROR,"relation %u deleted while still in use",save_relid);
19591963
}
19601964

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp