From: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: pgsql: Rename contrib module basic_archive to basic_wal_module |
Date: | 2023-01-25 18:17:50 |
Message-ID: | 20230125181750.GC1098626@nathanxps13 |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Wed, Jan 25, 2023 at 12:49:45PM -0500, Robert Haas wrote:
> On Wed, Jan 25, 2023 at 12:37 AM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>> Rename contrib module basic_archive to basic_wal_module
>
> FWIW, I find this new name much less clear than the old one.
>
> If we want to provide a basic_archive module and a basic_recovery
> module, that seems fine. Why merge them?
I'll admit I've been stewing on whether "WAL Modules" is the right name.
My first instinct was to simply call it "Archive and Recovery Modules,"
which is longer but (IMHO) clearer.
I wanted to merge basic_archive and basic_recovery because there's a decent
chunk of duplicated code. Perhaps that is okay, but I would rather just
have one test module. AFAICT the biggest reason to split it is because we
can't determine a good name. Maybe we could leave the name as
"basic_archive" since it deals with creating and recovering archive files.
--
Nathan Bossart
Amazon Web Services: https://aws.amazon.com
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2023-01-25 19:05:39 | Re: pgsql: Rename contrib module basic_archive to basic_wal_module |
Previous Message | Andres Freund | 2023-01-25 17:59:51 | pgsql: plpython: Avoid the need to redefine *printf macros |
From | Date | Subject | |
---|---|---|---|
Next Message | Israel Barth Rubio | 2023-01-25 18:27:04 | Re: Authentication fails for md5 connections if ~/.postgresql/postgresql.{crt and key} exist |
Previous Message | Justin Pryzby | 2023-01-25 18:00:20 | Re: Add LZ4 compression in pg_dump |