Re: pg_upgrade failing for 200+ million Large Objects
Magnus Hagander <magnus@hagander.net>
From: Magnus Hagander <magnus@hagander.net>
To: "Tharakan, Robins" <tharar@amazon.com>
Cc: Peter Eisentraut <peter.eisentraut@enterprisedb.com>, "pgsql-hackers@postgresql.org" <pgsql-hackers@postgresql.org>
Date: 2021-03-08T12:33:58Z
Lists: pgsql-hackers
On Mon, Mar 8, 2021 at 12:02 PM Tharakan, Robins <tharar@amazon.com> wrote: > > Thanks Peter. > > The original email [1] had some more context that somehow didn't get > associated with this recent email. Apologies for any confusion. Please take a look at your email configuration -- all your emails are lacking both References and In-reply-to headers, so every email starts a new thread, both for each reader and in the archives. It seems quite broken. It makes it very hard to follow. > In short, pg_resetxlog (and pg_resetwal) employs a magic constant [2] (for > both v9.6 as well as master) which seems to have been selected to force an > aggressive autovacuum as soon as the upgrade completes. Although that works > as planned, it narrows the window of Transaction IDs available for the > upgrade (before which XID wraparound protection kicks and aborts the > upgrade) to 146 Million. > > Reducing this magic constant allows a larger XID window, which is what the > patch is trying to do. With the patch, I was able to upgrade a cluster with > 500m Large Objects successfully (which otherwise reliably fails). In the > original email [1] I had also listed a few other possible workarounds, but > was unsure which would be a good direction to start working on.... thus this > patch to make a start. This still seems to just fix the symptoms and not the actual problem. What part of the pg_upgrade process is it that actually burns through that many transactions? Without looking, I would guess it's the schema reload using pg_dump/pg_restore and not actually pg_upgrade itself. This is a known issue in pg_dump/pg_restore. And if that is the case -- perhaps just running all of those in a single transaction would be a better choice? One could argue it's still not a proper fix, because we'd still have a huge memory usage etc, but it would then only burn 1 xid instead of 500M... AFAICT at a quick check, pg_dump in binary upgrade mode emits one lo_create() and one ALTER ... OWNER TO for each large object - so with 500M large objects that would be a billion statements, and thus a billion xids. And without checking, I'm fairly sure it doesn't load in a single transaction... -- Magnus Hagander Me: https://www.hagander.net/ Work: https://www.redpill-linpro.com/
Commits
Same data as JSON:
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Count individual SQL commands in pg_restore's --transaction-size mode.
- 81db073a2878 17.0 landed
- 0f1290521504 18.0 landed
-
Reduce number of commands dumpTableSchema emits for binary upgrade.
- b3f0e0503f33 18.0 landed
- 2fa989e6a340 17.0 landed
-
Invent --transaction-size option for pg_restore.
- 959b38d770ba 17.0 landed
-
Rearrange pg_dump's handling of large objects for better efficiency.
- a45c78e3284b 17.0 landed
-
Add temporal PRIMARY KEY and UNIQUE constraints
- 46a0cd4cefb4 17.0 cited
-
Fix typo and case in messages
- 7d7ef075d2b3 17.0 cited