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

Commit2168f37

Browse files
committed
Ignore CatalogSnapshot when checking COPY FREEZE prerequisites.
This restores the ability, essentially lost in commitffaa44c, to use COPY FREEZE underREPEATABLE READ isolation. Back-patch to 9.4, like that commit.Reviewed by Tom Lane.Discussion:https://postgr.es/m/CA+TgmoahWDm-7fperBxzU9uZ99LPMUmEpSXLTw9TmrOgzwnORw@mail.gmail.com
1 parentc3f43a7 commit2168f37

File tree

2 files changed

+23
-3
lines changed

2 files changed

+23
-3
lines changed

‎src/backend/commands/copy.c

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2392,13 +2392,25 @@ CopyFrom(CopyState cstate)
23922392
/*
23932393
* Optimize if new relfilenode was created in this subxact or one of its
23942394
* committed children and we won't see those rows later as part of an
2395-
* earlier scan or command.Thisensures that if thissubtransaction
2396-
* aborts then the frozen rows won't be visible after xact cleanup. Note
2395+
* earlier scan or command.The subxact testensures that if thissubxact
2396+
* aborts then the frozen rows won't be visible after xact cleanup.Note
23972397
* that the stronger test of exactly which subtransaction created it is
2398-
* crucial for correctness of this optimization.
2398+
* crucial for correctness of this optimization. The test for an earlier
2399+
* scan or command tolerates false negatives. FREEZE causes other sessions
2400+
* to see rows they would not see under MVCC, and a false negative merely
2401+
* spreads that anomaly to the current session.
23992402
*/
24002403
if (cstate->freeze)
24012404
{
2405+
/*
2406+
* Tolerate one registration for the benefit of FirstXactSnapshot.
2407+
* Scan-bearing queries generally create at least two registrations,
2408+
* though relying on that is fragile, as is ignoring ActiveSnapshot.
2409+
* Clear CatalogSnapshot to avoid counting its registration. We'll
2410+
* still detect ongoing catalog scans, each of which separately
2411+
* registers the snapshot it uses.
2412+
*/
2413+
InvalidateCatalogSnapshot();
24022414
if (!ThereAreNoPriorRegisteredSnapshots()|| !ThereAreNoReadyPortals())
24032415
ereport(ERROR,
24042416
(errcode(ERRCODE_INVALID_TRANSACTION_STATE),

‎src/backend/utils/time/snapmgr.c

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1645,6 +1645,14 @@ DeleteAllExportedSnapshotFiles(void)
16451645
FreeDir(s_dir);
16461646
}
16471647

1648+
/*
1649+
* ThereAreNoPriorRegisteredSnapshots
1650+
*Is the registered snapshot count less than or equal to one?
1651+
*
1652+
* Don't use this to settle important decisions. While zero registrations and
1653+
* no ActiveSnapshot would confirm a certain idleness, the system makes no
1654+
* guarantees about the significance of one registered snapshot.
1655+
*/
16481656
bool
16491657
ThereAreNoPriorRegisteredSnapshots(void)
16501658
{

0 commit comments

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