From: | Nitin Jadhav <nitinjadhavpostgres(at)gmail(dot)com> |
---|---|
To: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Michael Paquier <michael(at)paquier(dot)xyz>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Back patch of Remove durable_rename_excl() |
Date: | 2025-01-29 16:50:48 |
Message-ID: | CAMm1aWZCw6av9BJYFP0DAsTUs_FN2m-aMXvZ+5neFvoWsPrBsQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Thank you for providing the details. I agree to the concern about
breaking extensions that depend on the function. Gradually phasing it
out as versions become unsupported seems like a sensible approach.
Best Regards,
Nitin Jadhav
Azure Database for PostgreSQL
Microsoft
On Mon, Jan 27, 2025 at 8:43 PM Nathan Bossart <nathandbossart(at)gmail(dot)com> wrote:
>
> On Mon, Jan 27, 2025 at 09:47:22AM -0500, Tom Lane wrote:
> > Nitin Jadhav <nitinjadhavpostgres(at)gmail(dot)com> writes:
> >> I noticed that the corruption issue related to two hardlinks pointing
> >> to the same WAL file has been fixed in the master branch up to version
> >> 16 in commit [1]. As a result, the function durable_rename_excl()
> >> became unused and was removed in commit [2]. Since this corruption
> >> issue is occurring in older versions, commit [1] has been backported
> >> for versions 13 to 15 in commit [3]. However, I don't see the
> >> backporting for commit [2]. Is there a specific reason for this?
> >
> > Fear of breaking extensions that use the function, perhaps?
> > We don't like to break ABI in minor releases.
>
> Yup [0]. It'd be nice if we could get folks to stop using it, but that
> doesn't seem worth the ABI breakage, and from a couple of web searches,
> there doesn't seem to be much external use, anyway. IMHO letting it slowly
> phase out as versions go out of support is sufficient in this case.
>
> [0] https://postgr.es/m/20220418182336.GA2298576%40nathanxps13
>
> --
> nathan
From | Date | Subject | |
---|---|---|---|
Next Message | Rafael Thofehrn Castro | 2025-01-29 17:41:15 | Re: Proposal: Progressive explain |
Previous Message | Andres Freund | 2025-01-29 16:50:19 | Re: Windows CFBot is broken because ecpg dec_test.c error |