| From: | Henson Choi <assam258(at)gmail(dot)com> |
|---|---|
| To: | Tatsuo Ishii <ishii(at)postgresql(dot)org> |
| Cc: | zsolt(dot)parragi(at)percona(dot)com, sjjang112233(at)gmail(dot)com, vik(at)postgresfriends(dot)org, er(at)xs4all(dot)nl, jacob(dot)champion(at)enterprisedb(dot)com, david(dot)g(dot)johnston(at)gmail(dot)com, peter(at)eisentraut(dot)org, li(dot)evan(dot)chao(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: Row pattern recognition |
| Date: | 2026-05-13 04:15:20 |
| Message-ID: | CAAAe_zDJb6K7SdY7NHucKDZN_pZpOCbsA1Cy5_EpCZkqvdphKA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi Tatsuo,
Thanks for the review. Both points addressed in the attached
0013 v2; 0012/0014/0015 are unchanged from the 2026-05-12 set
and re-attached for convenience.
I think removing the exiting comment ("For WITH RECURSIVE, we rearrange
> the list elements...") is not appropriate as this explains subsequent
> process, which is not changed after the patch.
>
Agreed -- the original comment still describes the work-array
and tree-walker setup that follows, which the patch does not
touch. Restored it in place and moved the new ISO citation
comment to sit directly above the RPR rejection foreach so
each block is documented next to the code it explains.
> ERRCODE_FEATURE_NOT_SUPPORTED should be ERRCODE_SYNTAX_ERROR instead?
> IMO ERRCODE_FEATURE_NOT_SUPPORTED is used when the feature is defined
> by the standard but PostgreSQL just has not implemented yet. In this
> case the standard disllow RPR in recursive CTE.
Right, the standard explicitly prohibits this combination
rather than leaving it unimplemented. Switched to
ERRCODE_SYNTAX_ERROR. The .out files reference only the
ERROR message so the regress expected output is unchanged.
Thanks,
Henson
| Attachment | Content-Type | Size |
|---|---|---|
| nocfbot-0012-rpr_integration-B7-recursive-CTE-cases.txt | text/plain | 8.2 KB |
| nocfbot-0013-Reject-RPR-in-recursive-queries.txt | text/plain | 7.0 KB |
| nocfbot-0014-Enhance-README.rpr-per-Ishii-review.txt | text/plain | 5.7 KB |
| nocfbot-0015-Round-out-README.rpr-WindowAggState.txt | text/plain | 2.5 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2026-05-13 04:26:36 | Re: Fix unsafe PlannedStmt access in pg_stat_statements |
| Previous Message | Sami Imseih | 2026-05-13 03:45:07 | Re: [PATCH] Fix REPACK decoding worker not cleaned up on FATAL exit |