From: | David Steele <david(at)pgmasters(dot)net> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net>, Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: moving basebackup code to its own directory |
Date: | 2022-08-09 16:35:58 |
Message-ID: | 6543ccb3-347e-7833-df75-df5d50750cec@pgmasters.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 8/9/22 12:12, Magnus Hagander wrote:
> On Tue, Aug 9, 2022 at 6:08 PM Robert Haas <robertmhaas(at)gmail(dot)com
> <mailto:robertmhaas(at)gmail(dot)com>> wrote:
>
> Hi,
>
> I was thinking that it might make sense, to reduce clutter, to move
> *backup*.c from src/backend/replication to a new directory, perhaps
> src/backend/replication/backup or src/backend/backup.
>
> There's no particular reason we *have* to do this, but there are 21 C
> files in that directory and 11 of them are basebackup-related, so
> maybe it's time, especially because I think we might end up adding
> more basebackup-related stuff.
>
> Thoughts?
>
>
> Those 11 files are mostly your fault, of course ;)
>
> Anyway, I have no objection. If there'd been that many files, or plans
> to have it, in the beginning we probably would've put them in
> replication/basebackup or something like that from the beginning. I'm
> not sure how much it's worth doing wrt effects on backpatching etc, but
> if we're planning to add even more files in the future, the pain will
> just become bigger once we eventually do it...
There are big changes all around for PG15 so back-patching will be
complicated no matter what.
+1 from me and it would be great if we can get this into the PG15 branch
as well.
Regards,
-David
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2022-08-09 16:40:46 | Re: shared-memory based stats collector - v70 |
Previous Message | Robert Haas | 2022-08-09 16:34:46 | Re: moving basebackup code to its own directory |