| From: | Dent John <denty(at)QQdd(dot)eu> |
|---|---|
| To: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Cc: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Roman Pekar <roma(dot)pekar(at)gmail(dot)com> |
| Subject: | Re: [WIP] UNNEST(REFCURSOR): allowing SELECT to consume data from a REFCURSOR |
| Date: | 2019-12-14 12:09:09 |
| Message-ID: | 16F0596F-502E-4CD9-BCFC-7FA1BF63355E@QQdd.eu |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi folks,
I’ve made a revision of this patch.
The significant change is to access the Portal using Portal APIs rather than through SPI. It seems the persisted state necessary to survive being used to retrieve a row at a time inside an SRF just isn’t a good fit for SPI.
It turned out there was upstream machinery in the FunctionScan node that prevented Postgres being able to pipeline SRFs, even if they return ValuePerCall. So, in practice, this patch is of limited benefit without another patch that changes that behaviour (see [1]). Nevertheless, the code is independent so I’m submitting the two changes separately.
I’ll push this into the Jan commit fest.
denty.
[1] https://commitfest.postgresql.org/26/2372/ <https://commitfest.postgresql.org/26/2372/>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Fabien COELHO | 2019-12-14 12:49:08 | Re: psql's \watch is broken |
| Previous Message | Amit Kapila | 2019-12-14 06:29:43 | Re: logical decoding : exceeded maxAllocatedDescs for .spill files |