Re: Asynchronous MergeAppend

Matheus Alcantara <matheusssilv97@gmail.com>

View thread

From: "Matheus Alcantara" <matheusssilv97@gmail.com>

To: "Alexander Pyhalov" <a.pyhalov@postgrespro.ru>, "Matheus Alcantara" <matheusssilv97@gmail.com>

Cc: "Alena Rybakina" <a.rybakina@postgrespro.ru>, "Pgsql Hackers" <pgsql-hackers@postgresql.org>

Date: 2025-12-29T13:43:00Z

Lists: pgsql-hackers

Attachments

v11-0001-mark_async_capable-subpath-should-match-subplan.patch (text/plain)
v11-0002-MergeAppend-should-support-Async-Foreign-Scan-su.patch (text/plain)
v11-0003-Create-execAppend.c-to-avoid-duplicated-code-on-.patch (text/plain)

On Tue Dec 23, 2025 at 5:50 AM -03, Alexander Pyhalov wrote:
> I've looked through updated patch. Tested it (also with our fdw). 
> Overall looks good.
>
Thanks for testing.

> In execAppend.c there's still reference to as_valid_subplans. Also we 
> could perhaps use palloc0_array() in some more places, for example, for 
> for state->asyncrequests and state->asyncresults.
>
Fixed on the new attached version.

--
Matheus Alcantara
EDB: https://www.enterprisedb.com