diff options
| author | Peter Geoghegan <pg@bowt.ie> | 2022-11-23 11:10:06 -0800 |
|---|---|---|
| committer | Peter Geoghegan <pg@bowt.ie> | 2022-11-23 11:10:06 -0800 |
| commit | b6074846cebc33d752f1d9a66e5a9932f21ad177 (patch) | |
| tree | 33a2668556fa0ac3bb62c6759186588c3e4d1a0b /src/backend/commands/cluster.c | |
| parent | 02d647bbf0576ebb87f9dc24e1db4dd034f04048 (diff) | |
| download | postgresql-b6074846cebc33d752f1d9a66e5a9932f21ad177.tar.gz postgresql-b6074846cebc33d752f1d9a66e5a9932f21ad177.zip | |
Simplify vacuum_set_xid_limits() signature.
Pass VACUUM parameters (VacuumParams state) to vacuum_set_xid_limits()
directly, rather than passing most individual VacuumParams fields as
separate arguments.
Also make vacuum_set_xid_limits() output parameter symbol names match
those used by its vacuumlazy.c caller.
Author: Peter Geoghegan <pg@bowt.ie>
Discussion: https://postgr.es/m/CAH2-Wz=TE7gW5DgSahDkf0UEZigFGAoHNNN6EvSrdzC=Kn+hrA@mail.gmail.com
Diffstat (limited to 'src/backend/commands/cluster.c')
| -rw-r--r-- | src/backend/commands/cluster.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/backend/commands/cluster.c b/src/backend/commands/cluster.c index 3b78a2f100d..07e091bb87c 100644 --- a/src/backend/commands/cluster.c +++ b/src/backend/commands/cluster.c @@ -823,6 +823,7 @@ copy_table_data(Oid OIDNewHeap, Oid OIDOldHeap, Oid OIDOldIndex, bool verbose, Form_pg_class relform; TupleDesc oldTupDesc PG_USED_FOR_ASSERTS_ONLY; TupleDesc newTupDesc PG_USED_FOR_ASSERTS_ONLY; + VacuumParams params; TransactionId OldestXmin, FreezeXid; MultiXactId OldestMxact, @@ -914,7 +915,8 @@ copy_table_data(Oid OIDNewHeap, Oid OIDOldHeap, Oid OIDOldIndex, bool verbose, * Since we're going to rewrite the whole table anyway, there's no reason * not to be aggressive about this. */ - vacuum_set_xid_limits(OldHeap, 0, 0, 0, 0, &OldestXmin, &OldestMxact, + memset(¶ms, 0, sizeof(VacuumParams)); + vacuum_set_xid_limits(OldHeap, ¶ms, &OldestXmin, &OldestMxact, &FreezeXid, &MultiXactCutoff); /* |