Re: recovery modules

From: Nathan Bossart <nathandbossart(at)gmail(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Andres Freund <andres(at)anarazel(dot)de>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: recovery modules
Date: 2022-12-29 19:43:26
Message-ID: 20221229194326.GA296099@nathanxps13
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Here is a new patch set with the following changes:

* The restore_library, archive_cleanup_library, and recovery_end_library
parameters are merged into one parameter.

* restore_library can now be changed via SIGHUP. To provide a way for
modules to clean up when their callbacks are unloaded, I've introduced an
optional shutdown callback.

* Parameter misconfigurations are now always ERRORs. I'm less confident
that we can get by with just a WARNING now that restore_library can be
changed via SIGHUP, and this makes things more consistent with
archive_library/command.

--
Nathan Bossart
Amazon Web Services: https://aws.amazon.com

Attachment Content-Type Size
v2-0001-Move-the-code-to-restore-files-via-the-shell-to-a.patch text/x-diff 16.8 KB
v2-0002-Refactor-code-for-restoring-files-via-shell.patch text/x-diff 6.6 KB
v2-0003-Allow-recovery-via-loadable-modules.patch text/x-diff 51.9 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2022-12-29 20:00:34 Re: Avoiding unnecessary clog lookups while freezing
Previous Message Nathan Bossart 2022-12-29 18:52:14 Re: BUG #17717: Regression in vacuumdb (15 is slower than 10/11 and possible memory issue)