On Wed, Jan 25, 2023 at 04:50:22PM -0500, Tom Lane wrote:
> Nathan Bossart <nathandbossart@gmail.com> writes:
>> I wanted to merge basic_archive and basic_recovery because there's a decent
>> chunk of duplicated code.
>
> Would said code likely be duplicated into non-test uses of this feature?
> If so, maybe you ought to factor it out into a common location. I agree
> with Robert's point that basic_wal_module is a pretty content-free name.
I doubt it. The duplicated parts are things like _PG_init(), the check
hook for the GUC, and all the rest of the usual boilerplate stuff for
extensions (e.g., Makefile, meson.build). This module is small enough that
this probably makes up the majority of the code.
--
Nathan Bossart
Amazon Web Services: https://aws.amazon.com