Re: PG19: flat-copied RelOptInfos share extension_state with the original rel

From: Shubhankar Anand Kulkarni <shubhankar(dot)ak(at)zohocorp(dot)com>
To: "isho0628" <i(dot)sho(dot)0628(at)gmail(dot)com>
Cc: "pgsql-hackers" <pgsql-hackers(at)lists(dot)postgresql(dot)org>, "pgsql-hackers" <pgsql-hackers(at)postgresql(dot)org>, "robertmhaas" <robertmhaas(at)gmail(dot)com>, "guofenglinux" <guofenglinux(at)gmail(dot)com>
Subject: Re: PG19: flat-copied RelOptInfos share extension_state with the original rel
Date: 2026-10-13 10:37:35
Message-ID: 1a134d6cfd7.483591ea74208.6705674535977293651@zohocorp.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Oct 13, 2026 Sho Ito < mailto:i(dot)sho(dot)0628(at)gmail(dot)com > wrote:

> I reviewed v1 on master. All core regression tests pass.

> Before the patch, I reproduced the state-sharing warnings in

> mode 1 and the backend crash in mode 2. Both modes complete without

> warnings after the patch. I found no blocking issues.

Hi Sho,

Thanks for the review, and for reproducing it independently on master.

The commit-fest entry is https://commitfest.postgresql.org/patch/7436/ ,

and cfbot is green on all platforms.  If you're satisfied with it,

could you mark it Ready for Committer?

Since this touches both the extension-state API and

eager aggregation, are you OK with resetting the fields at each copy

site, or would you rather have a shared helper that does the flat copy

and the resets in one place? 

Also can we turn the reproducer into a src/test/modules test if you'd like regression coverage for it?

Regards,l

Shubhankar K.

Member Technical Staff

Zoho Corporation

From: Sho Ito <i(dot)sho(dot)0628(at)gmail(dot)com>
To: "Shubhankar Anand Kulkarni"<shubhankar(dot)ak(at)zohocorp(dot)com>
Cc: "pgsql-hackers"<pgsql-hackers(at)lists(dot)postgresql(dot)org>, "pgsql-hackers"<pgsql-hackers(at)postgresql(dot)org>, "robertmhaas"<robertmhaas(at)gmail(dot)com>, "guofenglinux"<guofenglinux(at)gmail(dot)com>
Date: Tue, 13 Oct 2026 04:01:52 +0530
Subject: Re: PG19: flat-copied RelOptInfos share extension_state with the original rel

On Mon, Oct 12, 2026 at 6:41 AM Shubhankar Anand Kulkarni < mailto:shubhankar(dot)ak(at)zohocorp(dot)com > wrote:

The attached patch resets the two fields at all four copy sites. 

Hi Shubhankar,

I reviewed v1 on master. All core regression tests pass.
Before the patch, I reproduced the state-sharing warnings in
mode 1 and the backend crash in mode 2. Both modes complete without
warnings after the patch. I found no blocking issues.

Thanks,
Sho Ito 

--
=======================================

Sho Ito (Sean)

E-mail:  mailto:i(dot)sho(dot)0628(at)gmail(dot)com

LinkedIn:  https://www.linkedin.com/in/sho-ito-0628/

GitHub:  https://github.com/sean0628/

=======================================

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jakub Wartak 2026-10-13 11:14:11 Re: [(known) BUG] DELETE/UPDATE more than one row in partitioned foreign table
Previous Message ZizhuanLiu X-MAN 2026-10-13 10:32:41 Re: Optimize MCV stats for sortable types and utilize sorted-order properties