Re: [PATCH] Add tests for src/backend/nodes/extensible.c

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

In response to

Browse pgsql-hackers by date

  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