From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Dave Page <dpage(at)pgadmin(dot)org>, Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Fix a typo in pg_rotate_logfile |
Date: | 2024-02-14 20:48:43 |
Message-ID: | A6967082-B1CB-491B-936C-5C7191BC58C9@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On 14 Feb 2024, at 19:51, Nathan Bossart <nathandbossart(at)gmail(dot)com> wrote:
>
> On Wed, Feb 14, 2024 at 10:04:49AM -0500, Tom Lane wrote:
>> Daniel Gustafsson <daniel(at)yesql(dot)se> writes:
>>> On 14 Feb 2024, at 11:35, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>> That said, pgAdmin III has been out of support for many years, and as
>>>> far as I know, it (and similarly old versions of EDB's PEM which was
>>>> based on it) were the only consumers of adminpack. I would not be sad
>>>> to see it removed entirely
>>
>>> Searching on Github and Debian Codesearch I cannot find any reference to anyone
>>> using any function from adminpack. With pgAdminIII being EOL it might be to
>>> remove it now rather than be on the hook to maintain it for another 5 years
>>> until v17 goes EOL. It'll still be around for years in V16->.
>>
>> Works for me.
>>
>>> Attached is a diff to show what it would look like to remove adminpack (catalog
>>> version bump omitted on purpose to avoid conflicts until commit).
>>
>> I don't see any references you missed, so +1.
>
> Seems reasonable to me, too.
Thanks! I'll put this in the next CF to keep it open for comments a bit
longer, but will close it early in the CF.
--
Daniel Gustafsson
From | Date | Subject | |
---|---|---|---|
Next Message | Melanie Plageman | 2024-02-14 21:02:29 | Re: index prefetching |
Previous Message | Noah Misch | 2024-02-14 19:55:43 | Re: common signal handler protection |