MIME-Version: 1.0
References: 
 <CAExHW5tYCE9QyCvVraKUeesKW5RTR+mrzsg3u64qSps-RPJR5A@mail.gmail.com>
 <CAAAe_zD3==epK=KufoBiLfogztpKbimxc498TUshTOQkkpgOjg@mail.gmail.com>
In-Reply-To: 
 <CAAAe_zD3==epK=KufoBiLfogztpKbimxc498TUshTOQkkpgOjg@mail.gmail.com>
From: Ashutosh Bapat <ashutosh.bapat.oss@gmail.com>
Date: Fri, 3 Apr 2026 11:07:03 +0530
Message-ID: 
 <CAExHW5s9aFkBDXuqf9np2asSRxm7vfuQAEbrXeHLOLCmwNjW4g@mail.gmail.com>
Subject: Re: SQL/PGQ: All properties reference
To: assam258@gmail.com
Cc: Junwang Zhao <zhjwpku@gmail.com>, Peter Eisentraut <peter@eisentraut.org>,
	pgsql-hackers <pgsql-hackers@postgresql.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CAExHW5s9aFkBDXuqf9np2asSRxm7vfuQAEbrXeHLOLCmwNjW4g%40mail.gmail.com>
Precedence: bulk

On Fri, Apr 3, 2026 at 7:24=E2=80=AFAM Henson Choi <assam258@gmail.com> wro=
te:
>
> Hi Ashutosh,
>
>> I am starting a new thread to discuss all properties reference feature
>> which was not committed with the main patch. [1]
>>
>> A <variable>.* is called all properties reference and it is allowed
>> only in COLUMNs clause. Interpreting subclause 9.2 and 9.3 together,
>> it expands to a list of graph property references <variable>.p1, ...
>> <variable>.pn where p1, ..., pn are the properties of the labels which
>> satisfy the label expression in the element pattern identified by
>> <variable>. The graph property references are added to the COLUMNs
>> clause in place of the all property reference, just like how <table>.*
>> expands in SELECT's targetlist.
>>
>> In the current implementation, we delay resolving graph property
>> references (<variable>.<property>) till the time query is generated
>> (generate_query_for_graph_path()). If we delay the all properties
>> reference till that time, we can not determine the data types and
>> names of the columns in the COLUMNs list. So we need to do that when
>> the COLUMNs clause is resolved. This means that the properties
>> associated with the labels needs to be resolved earlier. Since the
>> properties are not associated with labels directly but through the
>> elements, we need to find at least one element for every label in the
>> label expression. In brief, all the namespace resolution need to
>> happen before we transform COLUMNs clause. The patch rearranges the
>> code that way.
>
>
> I tried applying v20260318 on top of master to review it, but ran
> into merge conflicts in two files:
>
> - parse_graphtable.c
> - rewriteGraphTable.c
>
> The conflicts come from this commit that was added after the main PGQ
> commit (2f094e7ac69):
>
> - a0dd0702e46 Fix cross variable references in graph pattern causing
>   segfault
>
> Would it be possible to rebase the patch on the current master so
> I can review it cleanly?

I want to focus on resizable shared memory structures for PG 19 [1] in
whatever time remains until the feature freeze. If Peter feels that we
should get this in PG 19, I will rebase and finish the patch. I am ok,
if somebody else wants to rebase and finish this for PG 19 as well.
Sorry if this patch slips PG 19, but I will pick it up for PG 20, once
the branch opens.

[1] https://www.postgresql.org/message-id/CAExHW5vM1bneLYfg0wGeAa=3D52UiJ3z=
4vKd3AJ72X8Fw6k3KKrg@mail.gmail.com

--=20
Best Wishes,
Ashutosh Bapat