Re: non-exclusive backup cleanup is mildly broken - Mailing list pgsql-hackers

From Magnus Hagander
Subject Re: non-exclusive backup cleanup is mildly broken
Date
Msg-id CABUevExmWA3rrM7KEWkV2zT70cFST0d4ADmXv8t81r4-X-On1A@mail.gmail.com
Whole thread Raw
In response to Re: non-exclusive backup cleanup is mildly broken  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers

On Tue, Dec 17, 2019 at 7:05 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
Robert Haas <robertmhaas@gmail.com> writes:
> Perhaps I spoke too soon: I'm not sure whether Michael's comments
> amount to an objection. While I give him a chance to respond, here's
> an updated patch.

Took a quick look.  I agree that this seems a lot cleaner than the
alternative proposals.  I'd suggest however that the header comment
for do_pg_abort_backup could do with more work, perhaps along the
lines of "The odd-looking signature allows this to be registered
directly as a shmem_exit handler".

Personally I'd have kept the handler as a separate function that's just
a one-line wrapper around "void do_pg_abort_backup(bool emit_warning)".
We don't usually treat callbacks as functions to be also called in
their own right.  But if you don't want to do that, I'll settle for an
acknowledgement of the hack in the comment.

As would I, but I'm also fine with either of the two ways.
 
--

pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: allowing broader use of simplehash
Next
From: Magnus Hagander
Date:
Subject: Re: client auth docs seem to have devolved