Re: Row pattern recognition

From: Henson Choi <assam258(at)gmail(dot)com>
To: Tatsuo Ishii <ishii(at)postgresql(dot)org>, jian he <jian(dot)universality(at)gmail(dot)com>
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-11-13 02:20:39
Message-ID: CAAAe_zA7eFMiRRJAgHoaQ1S6YZWmMz7P=83kXnavg1jVq+NTyQ@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi Tatsuo,

> Attached is v54 RPR patches.

Attached are six incremental patches on top of v54, as nocfbot so
that cfbot keeps testing v54 itself. v54 still applies to master as
it is.

0001 fixes the cfbot failure of v54. Since "Add memory/disk usage
for Function Scan nodes in EXPLAIN" was committed on 11-10, EXPLAIN
ANALYZE prints a Storage line under Function Scan, and rpr_explain
fails on every platform. This updates rpr_explain.out and changes
nothing else.

0002-0006 take out of the series what is not specific to RPR:

0002 Remove the column alias list fix for table functions
0003 Remove the fix for columns of relations not printed in FROM
0004 Test the system column name a DEFINE clause reads on a view
column
0005 Remove the fix for columns that function result types gained
0006 Remove EXCLUDE TIES window tests

The three deparse fixes that 0002, 0003 and 0005 remove are bugs in
master as well, so I will submit them upstream on their own. If
they cause conflicts or need further changes in the series once
they are committed, I will send a follow-up patch then.

0004 rewrites the one rpr_base test that depended on the fix 0005
removes. 0006 drops two window.sql tests that have nothing to do
with RPR.

Since 0001 needs a new version anyway, this seems a good moment to
fold all six into v55. make check passes with all six applied.

Meanwhile, I am studying the planner side of the patch and auditing
v54 for defects. I will group what I find by severity and send it
in follow-up mails.

Best regards,
Henson

Attachment Content-Type Size
nocfbot-0001-update-rpr-explain-expected.txt text/plain 50.4 KB
nocfbot-0002-remove-tablefunc-alias-fix.txt text/plain 65.4 KB
nocfbot-0003-remove-unprinted-relation-fix.txt text/plain 11.4 KB
nocfbot-0004-system-column-test-on-view.txt text/plain 6.6 KB
nocfbot-0005-remove-function-grown-columns-fix.txt text/plain 20.2 KB
nocfbot-0006-remove-exclude-ties-window-tests.txt text/plain 2.9 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Chao Li 2026-11-13 02:20:57 Re: Distinguish EOF from a read error when loading the relcache init file
Previous Message Henson Choi 2026-11-13 02:19:05 Re: [DEPARSE] Parameters in SQL-standard function bodies do not read back