Feedback-ID: ie0a040ee:Fastmail
Message-ID: <42ba85f6-b38c-416e-97cb-595073e6956a@eisentraut.org>
Date: Wed, 8 Jul 2026 09:51:11 +0200
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: BUG #19542: Boolean syntax in GRAPH_TABLE
To: vaibhave postgres <postgresvaibhave@gmail.com>, h.lefebvre.69@gmail.com,
 pgsql-bugs@lists.postgresql.org
References: <19542-97a0785e900e2fde@postgresql.org>
 <CAM_eQjx8oO1CwcnYukKsTzzy-rcTn2vHdXk3jtgA91ZdoQ=4nw@mail.gmail.com>
Content-Language: en-US
From: Peter Eisentraut <peter@eisentraut.org>
In-Reply-To: 
 <CAM_eQjx8oO1CwcnYukKsTzzy-rcTn2vHdXk3jtgA91ZdoQ=4nw@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Archived-At: 
 <https://www.postgresql.org/message-id/42ba85f6-b38c-416e-97cb-595073e6956a%40eisentraut.org>
Precedence: bulk

On 05.07.26 10:20, vaibhave postgres wrote:
> Thanks for reporting.
> 
> The crash comes from replace_property_refs() (src/backend/rewrite/ 
> rewriteGraphTable.c) which rewrites the GraphPropertyRef nodes.
> 
> It calls expression_tree_mutator() directly on the top level node 
> instead of calling it's mutator (replace_property_refs_mutator()) which 
> treats GraphPropertyRef as a leaf & just copies it unchanged.
> 
>     ```
>     case T_GraphLabelRef:
>     case T_GraphPropertyRef:
>     case T_MergeSupportFunc:
> 
>         return copyObject(node);
> 
>     ```
> 
> 
> The fix is to call the mutator directly at the entry point.

Thanks, this has also been discovered in [0], and the fix there is the same.

[0]: 
https://www.postgresql.org/message-id/flat/20260630173053.51.noahmisch%40microsoft.com