| From: | Aleksander Alekseev <aleksander(at)tigerdata(dot)com> |
|---|---|
| To: | PostgreSQL Development <pgsql-hackers(at)postgresql(dot)org> |
| Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Jan Nidzwetzki <jan(at)planetscale(dot)com>, Rafia Sabih <rafia(dot)pghackers(at)gmail(dot)com> |
| Subject: | Re: [PATCH] Add tests for src/backend/nodes/extensible.c |
| Date: | 2026-08-25 13:51:15 |
| Message-ID: | CAJ7c6TMW_DY1QqQwc053P01cc6x3ZFS0UrSA3zuVPoF5G-L_pg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Jan, Michael,
Many thanks for all the great feedback! I applied all refactorings
except for the following.
> - Much better for me: add_int_reloption() to force a custom scan to
> return a number of rows you want with a CREATE TABLE .. WITH
> (repeat_count = N). That's doable once the library is loaded, and
> that should be a few extra lines of code. So let's add the reloption
> to control the scans rather than hardcoding it.
Perhaps I'm not fully understand the proposal but it doesn't seem to
work. Particularly after `git apply test.txt` I get:
```
TRAP: failed Assert("numoptions <= num_relopt_elems"), File:
"../src/backend/access/common/reloptions.c", Line: 2246, PID: 2077291
```
... in postmaster.log for a regular CREATE TABLE without the options.
Maybe you could clarify?
> - GUC to force a new per-query repeat_count.
Done this way.
--
Best regards,
Aleksander Alekseev
| Attachment | Content-Type | Size |
|---|---|---|
| test.txt | text/plain | 866 bytes |
| v4-0001-Add-test-module-for-src-backend-nodes-extensible..patch | text/x-patch | 46.4 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Pierre Forstmann | 2026-08-25 13:51:53 | Re: [PATCH] Remove obsolete tupDesc assignment in extended statistics |
| Previous Message | cca5507 | 2026-08-25 13:43:35 | Re: Walreceiver create temp slot more than once when timeline switch |