Re: Row pattern recognition
Henson Choi <assam258@gmail.com>
From: Henson Choi <assam258@gmail.com>
To: Tatsuo Ishii <ishii@postgresql.org>
Cc: vik@postgresfriends.org, er@xs4all.nl, jacob.champion@enterprisedb.com, david.g.johnston@gmail.com, peter@eisentraut.org, pgsql-hackers@postgresql.org
Date: 2026-03-01T11:02:38Z
Lists: pgsql-hackers
Attachments
- nocfbot-0011-define-cost.txt (text/plain)
- nocfbot-0012-qualified-refs.txt (text/plain)
Hi Tatsuo,
Attached are two more incremental patches (nocfbot-0011, nocfbot-0012)
on top of v43, continuing the nocfbot-0001..0010 series.
nocfbot-0011: Add RPR DEFINE expression cost to WindowAgg cost estimation
Revised version of your cost estimation patch [1]. Fixes the
`(char *)` cast and moves DEFINE cost outside the windowFuncs loop.
nocfbot-0012: Reject qualified column references in RPR DEFINE clause
Revised version of your qualified column reference patch [2].
Exposes `patternVarNames` via `ParseState.p_rpr_pattern_vars` to
distinguish pattern variable qualifiers ("not supported") from
FROM-clause range variable qualifiers ("not allowed").
Variables appearing only in DEFINE (not in PATTERN) are also
collected into `p_rpr_pattern_vars` so they get the "not supported"
message, but the RPR_VARID_MAX count check still applies only to
PATTERN variables.
[1]
https://www.postgresql.org/message-id/20260227.225456.33226875991025537.ishii@postgresql.org
[2]
https://www.postgresql.org/message-id/20260227.145539.1921177948671828231.ishii@postgresql.org
Attachment:
nocfbot-0011-define-cost.txt
nocfbot-0012-qualified-refs.txt
Best regards,
Henson
Commits
Same data as JSON:
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Adjust cross-version upgrade tests for seg_out() fix
- 3e3d7875e956 19 (unreleased) cited
-
Rationalize error comments in partition split/merge tests
- ecb2508aaf9b 19 (unreleased) cited
-
Add fast path for foreign key constraint checks
- 2da86c1ef9b5 19 (unreleased) cited
-
Fix assorted pretty-trivial memory leaks in the backend.
- e78d1d6d47dc 19 (unreleased) cited
-
Add temporal FOREIGN KEY contraints
- 89f908a6d0ac 18.0 cited
-
Add trailing commas to enum definitions
- 611806cd726f 17.0 cited
-
Remove obsolete executor cleanup code
- d060e921ea5a 17.0 cited