> Attached is the v47 patches for Row pattern recognition (SQL/RPR).
> - Add src/backend/executor/README.rpr (previously was in ExecRPR.c)
README.rpr is extremely useful for those who want to review the RPR
patches. I found a room to enhance the document. Attached is a small
patch tries to enhance README.rpr, on top of v47.
- Make "target audience" and "scope" of the README more descriptive.
- Add References (currently the SQL standards only)
- Add explanation of some abbreviations (NFA, AST)
- Add reference sections for absorption. Readers might not be familiar with "absorption"
- Add more fields to WindowAggState
- Add window framing rules with RPR
- 0014 Enhance README.rpr per Tatsuo Ishii's review
Doc-only. Applies your off-list enhancement patch
(2026-05-12) on top of v47 with no modification:
* Make "target audience" and "scope" more
descriptive, pointing readers to the SQL standard
(with Oracle / Trino manuals as alternatives)
* Spell out NFA and AST on first use
* Cross-reference "IV-5. Absorbability Analysis" and
"VIII-2. Solution: Context Absorption" from the
RPR_ELEM_ABSORBABLE_BRANCH flag description
* List nfaVisitedNWords, defineMatchStartDependent,
and nfaLastProcessedRow in V-3
* State the window framing rules that apply with RPR
* Add a References section (SQL standards)
- 0015 Round out README.rpr WindowAggState field coverage
Doc-only follow-up to 0014. Completes the V-3 field
list (adds nfaVisitedMinWord / nfaVisitedMaxWord; notes
that EXPLAIN ANALYZE instrumentation counters are
intentionally omitted, with a pointer to execnodes.h)
and mirrors the new fields in the Appendix B diagram,
which still reflected the pre-review field list.
Previously deferred items:
- B7 Recursive CTE XXX: addressed in 0012 (tests) + 0013
(rejection) per the standards citations you supplied.
Best regards,
Henson