Re: GRAPH_TABLE pattern WHERE clause is not coerced to boolean

From: Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>
To: Ewan Young <kdbase(dot)hack(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Peter Eisentraut <peter(at)eisentraut(dot)org>, Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>
Subject: Re: GRAPH_TABLE pattern WHERE clause is not coerced to boolean
Date: 2026-08-31 11:26:50
Message-ID: 3AE204BD-2682-4893-A100-88719E5CCDBB@gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On Aug 31, 2026, at 18:08, Ewan Young <kdbase(dot)hack(at)gmail(dot)com> wrote:
>
> Thanks for the review!
>
> On Mon, Aug 31, 2026 at 4:38 PM Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com> wrote:
>>
>>
>>
>>> On Aug 31, 2026, at 15:32, Ewan Young <kdbase(dot)hack(at)gmail(dot)com> wrote:
>>>
>>> Hi
>>>
>>> The WHERE clauses inside a GRAPH_TABLE pattern -- both the element-level
>>> one (MATCH (c IS customers WHERE ...)) and the graph-pattern-level one
>>> (MATCH ... WHERE ...) -- are transformed with a bare transformExpr()
>>> and never go through coerce_to_boolean(). So a WHERE clause of any
>>> type is accepted, and its raw datum is used as the qual:
>>>
>>> create table customers (id int primary key, name text);
>>> insert into customers values (1,'alice'),(2,null),(3,'carol');
>>> create property graph g
>>> vertex tables (customers key (id) label customer properties (id, name));
>>>
>>> select * from graph_table (g match (c is customer where c.name)
>>> columns (c.id));
>>> id
>>> ----
>>> 1
>>> 3
>>> (2 rows)
>>>
>>> EXPLAIN shows "Filter: name". The never-null text pointer is always
>>> taken as true, so the condition silently degenerates to roughly
>>> "name IS NOT NULL": the NULL-name row disappears with no error.
>>> Numeric quals are evaluated by bit pattern ("WHERE 1" is true,
>>> "WHERE 0" is false), and even "WHERE row(1,2)" is accepted. The same
>>> clause outside GRAPH_TABLE gives the usual
>>>
>>> ERROR: argument of WHERE must be type boolean, not type text
>>>
>>> The attached patch routes both sites through transformWhereClause(),
>>> like every other WHERE clause, and adds regression tests for the
>>> element-level and pattern-level cases. make check passes. The code is
>>> the same in REL_19_STABLE, so v19 is affected as well.
>>>
>>> --
>>> Regards,
>>> Ewan Young
>>> <v1-0001-Coerce-GRAPH_TABLE-pattern-WHERE-clauses-to-boolean.patch>
>>
>> The patch looks good to me. As this is a v19 bug, it might be worth noting in the Open Items list.
>
> Agreed -- I don't have wiki edit access though; could someone add it
> under "Open Issues"?
>

I just helped add the item.

Best regards,
--
Chao Li (Evan)
HighGo Software Co., Ltd.
https://www.highgo.com/

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Bertrand Drouvot 2026-08-31 11:27:07 Re: pg_upgrade: Test --check with a running source server
Previous Message Etsuro Fujita 2026-08-31 11:22:46 Re: Further cleanup related to statistics import support in postgres_fdw