Re: Segfault when restoring -Fd dump on current HEAD
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Alvaro Herrera <alvherre@2ndquadrant.com>
Cc: Dmitry Dolgov <9erthalion6@gmail.com>,
Michael Paquier <michael@paquier.xyz>,
Justin Pryzby <pryzby@telsasoft.com>,
Hubert Lubaczewski <depesz@depesz.com>,
pgsql-hackers mailing list <pgsql-hackers@postgresql.org>
Date: 2019-02-27T17:02:43Z
Lists: pgsql-hackers
Alvaro Herrera <alvherre@2ndquadrant.com> writes: > On 2019-Feb-27, Dmitry Dolgov wrote: >> But I hope there are no objections if I'll then submit the original >> changes with more consistent null handling separately to make decision >> about them more consciously. > I think we should save such a patch for whenever we next update the > archive version number, which could take a couple of years given past > history. I'm inclined to add a comment near K_VERS_SELF to remind > whoever next patches it. +1. This isn't an unreasonable cleanup idea, but being only a cleanup idea, it doesn't seem worth creating compatibility issues for. Let's wait till there is some more-pressing reason to change the archive format, and then fix this in the same release cycle. I'd also note that given what we've seen so far, there are going to be some slow-to-flush-out null pointer dereferencing bugs from this. I'm not really eager to introduce that towards the tail end of a devel cycle. regards, tom lane
Commits
-
pg_dump: store unused attribs as NULL instead of '\0'
- 7fcdb5e00214 12.0 landed
-
pg_dump: Fix ArchiveEntry handling of some empty values
- 19455c9f5606 12.0 landed