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

Commitbab3a71

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 parent86bc521 commitbab3a71

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
@@ -2394,13 +2394,25 @@ CopyFrom(CopyState cstate)
23942394
/*
23952395
* Optimize if new relfilenode was created in this subxact or one of its
23962396
* committed children and we won't see those rows later as part of an
2397-
* earlier scan or command.Thisensures that if thissubtransaction
2398-
* aborts then the frozen rows won't be visible after xact cleanup. Note
2397+
* earlier scan or command.The subxact testensures that if thissubxact
2398+
* aborts then the frozen rows won't be visible after xact cleanup.Note
23992399
* that the stronger test of exactly which subtransaction created it is
2400-
* crucial for correctness of this optimization.
2400+
* crucial for correctness of this optimization. The test for an earlier
2401+
* scan or command tolerates false negatives. FREEZE causes other sessions
2402+
* to see rows they would not see under MVCC, and a false negative merely
2403+
* spreads that anomaly to the current session.
24012404
*/
24022405
if (cstate->freeze)
24032406
{
2407+
/*
2408+
* Tolerate one registration for the benefit of FirstXactSnapshot.
2409+
* Scan-bearing queries generally create at least two registrations,
2410+
* though relying on that is fragile, as is ignoring ActiveSnapshot.
2411+
* Clear CatalogSnapshot to avoid counting its registration. We'll
2412+
* still detect ongoing catalog scans, each of which separately
2413+
* registers the snapshot it uses.
2414+
*/
2415+
InvalidateCatalogSnapshot();
24042416
if (!ThereAreNoPriorRegisteredSnapshots()|| !ThereAreNoReadyPortals())
24052417
ereport(ERROR,
24062418
(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