database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-17 06:41:24 +03:00
Code

Repurpose PROC_COPYABLE_FLAGS as PROC_XMIN_FLAGS

Browse Source 
This is a slight, convenient semantics change from what commit
0f0cfb494004 ("Fix parallel operations that prevent oldest xmin from
advancing") introduced that lets us simplify the coding in the one place
where it is used.

Backpatch to 13.  This is related to commit 6fea65508a1a ("Tighten
ComputeXidHorizons' handling of walsenders") rewriting the code site
where this is used, which has not yet been backpatched, but it may well
be in the future.

Reviewed-by: Masahiko Sawada <sawada.mshk@gmail.com>
Discussion: https://postgr.es/m/202204191637.eldwa2exvguw@alvherre.pgsql
This commit is contained in:
Alvaro Herrera 2022-05-19 16:20:32 +02:00
parent 5139db5563
commit 5fd0cccc11
No known key found for this signature in database
GPG Key ID: 1C20ACB9D5C564AE
2 changed files with 9 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/backend/storage/ipc/procarray.c
View File

@ -1916,12 +1916,13 @@ ProcArrayInstallRestoredXmin(TransactionId xmin, PGPROC *proc)
TransactionIdIsNormal(xid) && TransactionIdIsNormal(xid) &&
TransactionIdPrecedesOrEquals(xid, xmin)) TransactionIdPrecedesOrEquals(xid, xmin))
{ {
/* Install xmin */ /*
* Install xmin and propagate the vacuumFlags that affect how the
* value is interpreted by vacuum.
*/
MyPgXact->xmin = TransactionXmin = xmin; MyPgXact->xmin = TransactionXmin = xmin;
MyPgXact->vacuumFlags = (MyPgXact->vacuumFlags & ~PROC_XMIN_FLAGS) |
/* Flags being copied must be valid copy-able flags. */ (pgxact->vacuumFlags & PROC_XMIN_FLAGS);
Assert((pgxact->vacuumFlags & (~PROC_COPYABLE_FLAGS)) == 0);
MyPgXact->vacuumFlags = pgxact->vacuumFlags;
result = true; result = true;
} }

7
src/include/storage/proc.h
View File

@ -63,11 +63,10 @@ struct XidCache
(PROC_IN_VACUUM | PROC_IN_ANALYZE | PROC_VACUUM_FOR_WRAPAROUND) (PROC_IN_VACUUM | PROC_IN_ANALYZE | PROC_VACUUM_FOR_WRAPAROUND)
/* /*
* Flags that are valid to copy from another proc, the parallel leader * Xmin-related flags. Make sure any flags that affect how the process' Xmin
* process in practice. Currently, a flag that is set during parallel * value is interpreted by VACUUM are included here.
* vacuum is allowed.
*/ */
#define PROC_COPYABLE_FLAGS (PROC_IN_VACUUM) #define PROC_XMIN_FLAGS (PROC_IN_VACUUM)
/* /*
* We allow a small number of "weak" relation locks (AccessShareLock, * We allow a small number of "weak" relation locks (AccessShareLock,