| From: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
|---|---|
| To: | Andres Freund <andres(at)anarazel(dot)de> |
| Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Nitin Motiani <nitinmotiani(at)google(dot)com>, Hannu Krosing <hannuk(at)google(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: pg_upgrade: transfer pg_largeobject_metadata's files when possible |
| Date: | 2026-02-12 21:59:24 |
| Message-ID: | aY5NPLeup3VVucsb@nathan |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Feb 12, 2026 at 04:46:30PM -0500, Andres Freund wrote:
> On 2026-02-11 15:00:51 -0600, Nathan Bossart wrote:
>> This was a little more painful than I expected, but this seems to be what
>> is required to allow COPY-ing pg_largeobject_metadata during binary
>> upgrades from < v12.
>
> Nice!
Thanks for looking.
> Not really the fault of this patch, but it seems somewhat grotty that this has
> binary upgrade specific code in this place. I was certainly confused when
> first trying to use pg_dump in binary upgrade mode with large objects, because
> no data was dumped when using plain text mode, which is what I had been using
> for simplicity...
Yeah, this has been an annoyance while hacking in this area. I haven't
looked into what it'd take to fix it, though.
> I guess you could instead generate a COPY using WITH OIDS. But it's probably
> not worth having that path, given we already need to support COPY (SELECT ..).
Ah, I forgot all about that COPY option...
> as we'd just include the oid column without needing to somehow include it in
> the attribute list.
>
> [...]
>
> I guess WITH OIDs also would avoid the need for this.
Will give it a try.
--
nathan
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Bruce Momjian | 2026-02-12 22:12:02 | Re: Support logical replication of DDLs |
| Previous Message | Andres Freund | 2026-02-12 21:46:30 | Re: pg_upgrade: transfer pg_largeobject_metadata's files when possible |