Handle WindowClause.runCondition in tree walker/mutator functions.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 10 Jan 2024 18:36:34 +0000 (13:36 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 10 Jan 2024 18:36:34 +0000 (13:36 -0500)
commitee955321869e46289d11b0c77255c74d62caf3a5
treef9d50341a0ad6754465d5d6cfd75d7885511b525
parent65ee2ed0d0da6dd45853b050b4d59634d64479bd
Handle WindowClause.runCondition in tree walker/mutator functions.

Commit 9d9c02ccd, which added the notion of a "run condition" for
window functions, neglected to teach nodeFuncs.c to process the new
field.  Remarkably, that doesn't seem to have had any ill effects
before we invented Var.varnullingrels, but now it can cause visible
failures in join-removal scenarios.

I have no faith that there's not reachable problems in v15 too,
so back-patch the code change to v15 where 9d9c02ccd came in.
The test case seems irrelevant to v15, though.

Per bug #18277 from Zuming Jiang.  Diagnosis and patch by
Richard Guo.

Discussion: https://postgr.es/m/18277-089ead83b329a2fd@postgresql.org
src/backend/nodes/nodeFuncs.c
src/test/regress/expected/window.out
src/test/regress/sql/window.sql