| From: | Tatsuo Ishii <ishii(at)postgresql(dot)org> |
|---|---|
| To: | assam258(at)gmail(dot)com |
| Cc: | 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, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: Row pattern recognition |
| Date: | 2026-03-02 05:18:23 |
| Message-ID: | 20260302.141823.129040280886273031.ishii@postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
>> Thanks. I will try them out. Once they look good, shall I release v44
>> patch sets? Or do you have a plan to add more patches?
>
> The NFA executor code looks stable to me at this point. I do have some
> additional optimization ideas around absorption, but I think it's better
> new features should be added more carefully, or deferred until after the
> commit.
>
> Please go ahead and release v44. Having a fresh patch set will also
> make it easier to proceed with the next round of test modifications
> and improvements.
Your patches look good. Attached are the v44 pathes, with minor fixes
to docs (mention that reluctant quantifiers are supported, exclusion
and empty PATTERN are not supported). Also makes clear the range of n
and m in {n, m} and other regular expression.
---- Major differences from v43 patches ----
- Implement reluctant quantifiers
- Disallow using PREV()/NEXT() in other than the DEFINE caluse
- Reject qualied column references in the DEFINE clause
- Fix explain test failures on 32bit platforms
- Fix and cleanup the NFA engine
- Fix docs that reluctant quantifiers are supported, exclusion and
empty PATTERN are not supported.
- Some code refactorings
Best regards,
--
Tatsuo Ishii
SRA OSS K.K.
English: http://www.sraoss.co.jp/index_en/
Japanese:http://www.sraoss.co.jp
| Attachment | Content-Type | Size |
|---|---|---|
| v44-0001-Row-pattern-recognition-patch-for-raw-parser.patch | application/octet-stream | 31.4 KB |
| v44-0002-Row-pattern-recognition-patch-parse-analysis.patch | application/octet-stream | 30.6 KB |
| v44-0003-Row-pattern-recognition-patch-rewriter.patch | application/octet-stream | 5.8 KB |
| v44-0004-Row-pattern-recognition-patch-planner.patch | application/octet-stream | 68.8 KB |
| v44-0005-Row-pattern-recognition-patch-executor-and-comma.patch | application/octet-stream | 103.7 KB |
| v44-0006-Row-pattern-recognition-patch-docs.patch | application/octet-stream | 16.3 KB |
| v44-0007-Row-pattern-recognition-patch-tests.patch | application/octet-stream | 943.7 KB |
| v44-0008-Row-pattern-recognition-patch-typedefs.list.patch | application/octet-stream | 1.1 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Corey Huinker | 2026-03-02 05:18:36 | Re: Add starelid, attnum to pg_stats and leverage this in pg_dump |
| Previous Message | jian he | 2026-03-02 04:57:20 | Refactor handling of "-only" options in pg_dump, pg_restore |