MIME-Version: 1.0
From: SATYANARAYANA NARLAPURAM <satyanarlapuram@gmail.com>
Date: Tue, 21 Apr 2026 08:56:44 -0700
Message-ID: 
 <CAHg+QDc97WFTSkXg=g_ZAH8GnY2gJrvq72cs+YjqEAuZgXnkAQ@mail.gmail.com>
Subject: [Patch]Add Graph* node support to expression_tree_mutator
To: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>,
	Ashutosh Bapat <ashutosh.bapat.oss@gmail.com>
Content-Type: multipart/mixed; boundary="000000000000f8666d064ffa764f"
Archived-At: 
 <https://www.postgresql.org/message-id/CAHg%2BQDc97WFTSkXg%3Dg_ZAH8GnY2gJrvq72cs%2BYjqEAuZgXnkAQ%40mail.gmail.com>
Precedence: bulk

--000000000000f8666d064ffa764f
Content-Type: multipart/alternative; boundary="000000000000f8666b064ffa764d"

--000000000000f8666b064ffa764d
Content-Type: text/plain; charset="UTF-8"

Hi hackers,

expression_tree_mutator_impl() was missing case handlers for
T_GraphPattern, T_GraphElementPattern, and T_GraphPropertyRef.
The corresponding expression_tree_walker_impl() already handled
all three node types, but the mutator did not, causing an
"unrecognized node type: 106" error whenever a GRAPH_TABLE
subquery appeared in a HAVING clause.

SELECT 1 FROM hv GROUP BY id
HAVING (SELECT COUNT(*) FROM GRAPH_TABLE(gh MATCH (a) COLUMNS (a.id AS x)))
> 0;
SET
ERROR:  unrecognized node type: 106

Attached a patch to address this.

Thanks,
Satya

--000000000000f8666b064ffa764d
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">Hi hackers,<div><br></div><div>expression_tree_mutator_imp=
l() was missing case handlers for<br>T_GraphPattern, T_GraphElementPattern,=
 and T_GraphPropertyRef.<br>The corresponding expression_tree_walker_impl()=
 already handled<br>all three node types, but the mutator did not, causing =
an<br>&quot;unrecognized node type: 106&quot; error whenever a GRAPH_TABLE<=
br>subquery appeared in a HAVING clause.</div><div><br></div><div>SELECT 1 =
FROM hv GROUP BY id<br>HAVING (SELECT COUNT(*) FROM GRAPH_TABLE(gh MATCH (a=
) COLUMNS (<a href=3D"http://a.id">a.id</a> AS x))) &gt; 0;<br>SET<br>ERROR=
: =C2=A0unrecognized node type: 106</div><div><br></div><div>Attached a pat=
ch to address this.</div><div><br></div><div>Thanks,</div><div>Satya</div><=
/div>

--000000000000f8666b064ffa764d--

--000000000000f8666d064ffa764f
Content-Type: application/octet-stream;
	name="0001-Add-Graph-node-support-to-expression_tree_mutator.patch"
Content-Disposition: attachment;
	filename="0001-Add-Graph-node-support-to-expression_tree_mutator.patch"
Content-Transfer-Encoding: base64
Content-ID: <f_mo8t3jkq0>
X-Attachment-Id: f_mo8t3jkq0

RnJvbSAwMDM0MGZhYTNiZTY0NTQxZjZmYWMyYmNhNTU4ODBmNjIxZjk0MGQwIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBzYXR5YW5hcmF5YW5hIG5hcmxhcHVyYW0gPHNhdHlhbmFybGFw
dXJhbUBnbWFpbC5jb20+CkRhdGU6IFR1ZSwgMjEgQXByIDIwMjYgMTQ6MzM6MzMgKzAwMDAKU3Vi
amVjdDogW1BBVENIXSBBZGQgR3JhcGgqIG5vZGUgc3VwcG9ydCB0byBleHByZXNzaW9uX3RyZWVf
bXV0YXRvcgoKZXhwcmVzc2lvbl90cmVlX211dGF0b3JfaW1wbCgpIHdhcyBtaXNzaW5nIGNhc2Ug
aGFuZGxlcnMgZm9yClRfR3JhcGhQYXR0ZXJuLCBUX0dyYXBoRWxlbWVudFBhdHRlcm4sIGFuZCBU
X0dyYXBoUHJvcGVydHlSZWYuClRoZSBjb3JyZXNwb25kaW5nIGV4cHJlc3Npb25fdHJlZV93YWxr
ZXJfaW1wbCgpIGFscmVhZHkgaGFuZGxlZAphbGwgdGhyZWUgbm9kZSB0eXBlcywgYnV0IHRoZSBt
dXRhdG9yIGRpZCBub3QsIGNhdXNpbmcgYW4KInVucmVjb2duaXplZCBub2RlIHR5cGU6IDEwNiIg
ZXJyb3Igd2hlbmV2ZXIgYSBHUkFQSF9UQUJMRQpzdWJxdWVyeSBhcHBlYXJlZCBpbiBhIEhBVklO
RyBjbGF1c2UuCgpUaGUgZml4IGFkZHMgdGhyZWUgY2FzZSBoYW5kbGVycyBtaXJyb3JpbmcgdGhl
IHdhbGtlcjoKLSBUX0dyYXBoUHJvcGVydHlSZWY6IGxlYWYgbm9kZSwgRkxBVENPUFkgb25seQot
IFRfR3JhcGhFbGVtZW50UGF0dGVybjogRkxBVENPUFkgKyBNVVRBVEUgc3ViZXhwciwgd2hlcmVD
bGF1c2UKLSBUX0dyYXBoUGF0dGVybjogRkxBVENPUFkgKyBNVVRBVEUgcGF0aF9wYXR0ZXJuX2xp
c3QsIHdoZXJlQ2xhdXNlCi0tLQogc3JjL2JhY2tlbmQvbm9kZXMvbm9kZUZ1bmNzLmMgICAgICAg
ICAgICAgfCAzMSArKysrKysrKysrKysrKysrKysrKysrKwogc3JjL3Rlc3QvcmVncmVzcy9leHBl
Y3RlZC9ncmFwaF90YWJsZS5vdXQgfCAxNSArKysrKysrKysrKwogc3JjL3Rlc3QvcmVncmVzcy9z
cWwvZ3JhcGhfdGFibGUuc3FsICAgICAgfCAgOSArKysrKysrCiAzIGZpbGVzIGNoYW5nZWQsIDU1
IGluc2VydGlvbnMoKykKCmRpZmYgLS1naXQgYS9zcmMvYmFja2VuZC9ub2Rlcy9ub2RlRnVuY3Mu
YyBiL3NyYy9iYWNrZW5kL25vZGVzL25vZGVGdW5jcy5jCmluZGV4IGMwYjg4MGVjLi5jMThmMmIw
ZSAxMDA2NDQKLS0tIGEvc3JjL2JhY2tlbmQvbm9kZXMvbm9kZUZ1bmNzLmMKKysrIGIvc3JjL2Jh
Y2tlbmQvbm9kZXMvbm9kZUZ1bmNzLmMKQEAgLTM4MTAsNiArMzgxMCwzNyBAQCBleHByZXNzaW9u
X3RyZWVfbXV0YXRvcl9pbXBsKE5vZGUgKm5vZGUsCiAJCQkJcmV0dXJuIChOb2RlICopIG5ld25v
ZGU7CiAJCQl9CiAJCQlicmVhazsKKwkJY2FzZSBUX0dyYXBoUHJvcGVydHlSZWY6CisJCQl7CisJ
CQkJR3JhcGhQcm9wZXJ0eVJlZiAqbmV3bm9kZTsKKworCQkJCUZMQVRDT1BZKG5ld25vZGUsIG5v
ZGUsIEdyYXBoUHJvcGVydHlSZWYpOworCQkJCS8qIGxlYWYgbm9kZSwgbm8gZXhwcmVzc2lvbiBz
dWJub2RlcyAqLworCQkJCXJldHVybiAoTm9kZSAqKSBuZXdub2RlOworCQkJfQorCQkJYnJlYWs7
CisJCWNhc2UgVF9HcmFwaEVsZW1lbnRQYXR0ZXJuOgorCQkJeworCQkJCUdyYXBoRWxlbWVudFBh
dHRlcm4gKmdlcCA9IChHcmFwaEVsZW1lbnRQYXR0ZXJuICopIG5vZGU7CisJCQkJR3JhcGhFbGVt
ZW50UGF0dGVybiAqbmV3bm9kZTsKKworCQkJCUZMQVRDT1BZKG5ld25vZGUsIGdlcCwgR3JhcGhF
bGVtZW50UGF0dGVybik7CisJCQkJTVVUQVRFKG5ld25vZGUtPnN1YmV4cHIsIGdlcC0+c3ViZXhw
ciwgTGlzdCAqKTsKKwkJCQlNVVRBVEUobmV3bm9kZS0+d2hlcmVDbGF1c2UsIGdlcC0+d2hlcmVD
bGF1c2UsIE5vZGUgKik7CisJCQkJcmV0dXJuIChOb2RlICopIG5ld25vZGU7CisJCQl9CisJCQli
cmVhazsKKwkJY2FzZSBUX0dyYXBoUGF0dGVybjoKKwkJCXsKKwkJCQlHcmFwaFBhdHRlcm4gKmdw
ID0gKEdyYXBoUGF0dGVybiAqKSBub2RlOworCQkJCUdyYXBoUGF0dGVybiAqbmV3bm9kZTsKKwor
CQkJCUZMQVRDT1BZKG5ld25vZGUsIGdwLCBHcmFwaFBhdHRlcm4pOworCQkJCU1VVEFURShuZXdu
b2RlLT5wYXRoX3BhdHRlcm5fbGlzdCwgZ3AtPnBhdGhfcGF0dGVybl9saXN0LCBMaXN0ICopOwor
CQkJCU1VVEFURShuZXdub2RlLT53aGVyZUNsYXVzZSwgZ3AtPndoZXJlQ2xhdXNlLCBOb2RlICop
OworCQkJCXJldHVybiAoTm9kZSAqKSBuZXdub2RlOworCQkJfQorCQkJYnJlYWs7CiAJCWRlZmF1
bHQ6CiAJCQllbG9nKEVSUk9SLCAidW5yZWNvZ25pemVkIG5vZGUgdHlwZTogJWQiLAogCQkJCSAo
aW50KSBub2RlVGFnKG5vZGUpKTsKZGlmZiAtLWdpdCBhL3NyYy90ZXN0L3JlZ3Jlc3MvZXhwZWN0
ZWQvZ3JhcGhfdGFibGUub3V0IGIvc3JjL3Rlc3QvcmVncmVzcy9leHBlY3RlZC9ncmFwaF90YWJs
ZS5vdXQKaW5kZXggYjU3OWUzZGYuLjVlNTdkMjQ4IDEwMDY0NAotLS0gYS9zcmMvdGVzdC9yZWdy
ZXNzL2V4cGVjdGVkL2dyYXBoX3RhYmxlLm91dAorKysgYi9zcmMvdGVzdC9yZWdyZXNzL2V4cGVj
dGVkL2dyYXBoX3RhYmxlLm91dApAQCAtMTAyMiw0ICsxMDIyLDE5IEBAIFNFTEVDVCBzbmFtZSwg
ZG5hbWUgRlJPTSBHUkFQSF9UQUJMRSAoZzEgTUFUQ0ggKHNyYyktPihkZXN0KSBXSEVSRSBzcmMu
dnByb3AxID4KIEVSUk9SOiAgc3VicXVlcmllcyB3aXRoaW4gR1JBUEhfVEFCTEUgcmVmZXJlbmNl
IGFyZSBub3Qgc3VwcG9ydGVkCiBTRUxFQ1Qgc25hbWUsIGRuYW1lIEZST00gR1JBUEhfVEFCTEUg
KGcxIE1BVENIIChzcmMpLT4oZGVzdCkgV0hFUkUgb3V0X2RlZ3JlZShzcmMudm5hbWUpID4gKFNF
TEVDVCBtYXgob3V0X2RlZ3JlZShubmFtZSkpIEZST00gR1JBUEhfVEFCTEUgKGcxIE1BVENIIChu
b2RlKSBDT0xVTU5TIChub2RlLnZuYW1lIEFTIG5uYW1lKSkpIENPTFVNTlMoc3JjLnZuYW1lIEFT
IHNuYW1lLCBkZXN0LnZuYW1lIEFTIGRuYW1lKSk7CiBFUlJPUjogIHN1YnF1ZXJpZXMgd2l0aGlu
IEdSQVBIX1RBQkxFIHJlZmVyZW5jZSBhcmUgbm90IHN1cHBvcnRlZAorLS0gR1JBUEhfVEFCTEUg
c3VicXVlcnkgaW4gSEFWSU5HIGNsYXVzZQorU0VMRUNUIHNyYy52bmFtZSwgY291bnQoKikKKyAg
RlJPTSB2MSBBUyBzcmMKKyAgR1JPVVAgQlkgc3JjLnZuYW1lCisgIEhBVklORyBjb3VudCgqKSA+
PSAoU0VMRUNUIGNvdW50KCopCisgICAgICAgICAgICAgICAgICAgICAgICBGUk9NIEdSQVBIX1RB
QkxFIChnMSBNQVRDSCAoYSBJUyB2bDEpCisgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICBDT0xVTU5TIChhLnZuYW1lIEFTIG4pKQorICAgICAgICAgICAgICAgICAgICAg
ICBXSEVSRSBuID0gc3JjLnZuYW1lKTsKKyB2bmFtZSB8IGNvdW50IAorLS0tLS0tLSstLS0tLS0t
CisgdjEzICAgfCAgICAgMQorIHYxMiAgIHwgICAgIDEKKyB2MTEgICB8ICAgICAxCisoMyByb3dz
KQorCiAtLSBsZWF2ZSB0aGUgb2JqZWN0cyBiZWhpbmQgZm9yIHBnX3VwZ3JhZGUvcGdfZHVtcCB0
ZXN0cwpkaWZmIC0tZ2l0IGEvc3JjL3Rlc3QvcmVncmVzcy9zcWwvZ3JhcGhfdGFibGUuc3FsIGIv
c3JjL3Rlc3QvcmVncmVzcy9zcWwvZ3JhcGhfdGFibGUuc3FsCmluZGV4IDRmZjk4ODE3Li4zZDUy
MjRkMyAxMDA2NDQKLS0tIGEvc3JjL3Rlc3QvcmVncmVzcy9zcWwvZ3JhcGhfdGFibGUuc3FsCisr
KyBiL3NyYy90ZXN0L3JlZ3Jlc3Mvc3FsL2dyYXBoX3RhYmxlLnNxbApAQCAtNTgyLDQgKzU4Miwx
MyBAQCBTRUxFQ1QgKiBGUk9NIGN1c3RvbWVycyBjbyBXSEVSRSBjby5jdXN0b21lcl9pZCA9IChT
RUxFQ1QgY3VzdG9tZXJfaWQgRlJPTSBHUkFQSAogU0VMRUNUIHNuYW1lLCBkbmFtZSBGUk9NIEdS
QVBIX1RBQkxFIChnMSBNQVRDSCAoc3JjKS0+KGRlc3QpIFdIRVJFIHNyYy52cHJvcDEgPiAoU0VM
RUNUIG1heCh2MS52cHJvcDEpIEZST00gdjEpIENPTFVNTlMoc3JjLnZuYW1lIEFTIHNuYW1lLCBk
ZXN0LnZuYW1lIEFTIGRuYW1lKSk7CiBTRUxFQ1Qgc25hbWUsIGRuYW1lIEZST00gR1JBUEhfVEFC
TEUgKGcxIE1BVENIIChzcmMpLT4oZGVzdCkgV0hFUkUgb3V0X2RlZ3JlZShzcmMudm5hbWUpID4g
KFNFTEVDVCBtYXgob3V0X2RlZ3JlZShubmFtZSkpIEZST00gR1JBUEhfVEFCTEUgKGcxIE1BVENI
IChub2RlKSBDT0xVTU5TIChub2RlLnZuYW1lIEFTIG5uYW1lKSkpIENPTFVNTlMoc3JjLnZuYW1l
IEFTIHNuYW1lLCBkZXN0LnZuYW1lIEFTIGRuYW1lKSk7CiAKKy0tIEdSQVBIX1RBQkxFIHN1YnF1
ZXJ5IGluIEhBVklORyBjbGF1c2UKK1NFTEVDVCBzcmMudm5hbWUsIGNvdW50KCopCisgIEZST00g
djEgQVMgc3JjCisgIEdST1VQIEJZIHNyYy52bmFtZQorICBIQVZJTkcgY291bnQoKikgPj0gKFNF
TEVDVCBjb3VudCgqKQorICAgICAgICAgICAgICAgICAgICAgICAgRlJPTSBHUkFQSF9UQUJMRSAo
ZzEgTUFUQ0ggKGEgSVMgdmwxKQorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgQ09MVU1OUyAoYS52bmFtZSBBUyBuKSkKKyAgICAgICAgICAgICAgICAgICAgICAgV0hF
UkUgbiA9IHNyYy52bmFtZSk7CisKIC0tIGxlYXZlIHRoZSBvYmplY3RzIGJlaGluZCBmb3IgcGdf
dXBncmFkZS9wZ19kdW1wIHRlc3RzCi0tIAoyLjQzLjAKCg==
--000000000000f8666d064ffa764f--