Re: Reducing connection overhead in pg_upgrade compat check phase

On 07.02.24 14:25, Daniel Gustafsson wrote:
>> On 6 Feb 2024, at 17:47, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
>>
>>> On 6 Feb 2024, at 17:32, Nathan Bossart <nathandbossart(at)gmail(dot)com> wrote:
>>>
>>> On Fri, Feb 02, 2024 at 12:18:25AM +0530, vignesh C wrote:
>>>> With no update to the thread and the patch still not applying I'm
>>>> marking this as returned with feedback. Please feel free to resubmit
>>>> to the next CF when there is a new version of the patch.
>>>
>>> IMHO this patch is worth trying to get into v17. I'd be happy to take it
>>> forward if Daniel does not intend to work on it.
>>
>> I actually had the same thought yesterday and spent some time polishing and
>> rebasing it. I'll post an updated rebase shortly with the hopes of getting it
>> committed this week.
>
> Attached is a v11 rebased over HEAD with some very minor tweaks. Unless there
> are objections I plan to go ahead with this version this week.

A few more quick comments:

I think the .report_text assignments also need a gettext_noop(), like
the .status assignments.

The type DataTypesUsageChecks is only used in check.c, so doesn't need
to be in pg_upgrade.h.

Idea for further improvement: Might be nice if the
DataTypesUsageVersionCheck struct also included the applicable version
information, so the additional checks in version.c would no longer be
necessary.