Re: Row pattern recognition

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

In response to

Browse pgsql-hackers by date

  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