| From: | Alexander Pyhalov <a(dot)pyhalov(at)postgrespro(dot)ru> |
|---|---|
| To: | Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Cc: | Alena Rybakina <a(dot)rybakina(at)postgrespro(dot)ru>, Matheus Alcantara <matheusssilv97(at)gmail(dot)com> |
| Subject: | Re: Asynchronous MergeAppend |
| Date: | 2026-03-02 06:44:46 |
| Message-ID: | 554a73f7ffea8b22b3f81a4804b5fc34@postgrespro.ru |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Alexander Pyhalov писал(а) 2026-02-12 10:08:
> Alexander Pyhalov писал(а) 2025-12-30 22:04:
>> Matheus Alcantara писал(а) 2025-12-30 18:04:
>>> On Tue Dec 30, 2025 at 10:15 AM -03, Alexander Pyhalov wrote:
>>>> Looks good. What do you think about
>>>> classify_matching_subplans_common()?
>>>> Should it stay where it is or should we hide it to
>>>> src/include/executor/execAppend.h ?
>>>>
>>> Yeah, sounds better to me to move classify_matching_subplans_common
>>> to
>>> execAppend.h since it very specific for the MergeAppend and Append
>>> execution. I've declared the function as static inline on
>>> execAppend.h
>>> but I'm not sure if it's the best approach.
>>>
>>> I've also wrote a proper commit message for this new version.
>>
>> Looks good to me.
>
> Hi.
> Rebased patches over fresh master.
Hi.
Rebased patches over fresh master.
--
Best regards,
Alexander Pyhalov,
Postgres Professional
| Attachment | Content-Type | Size |
|---|---|---|
| v14-0001-mark_async_capable-subpath-should-match-subplan.patch | text/x-diff | 2.4 KB |
| v14-0002-MergeAppend-should-support-Async-Foreign-Scan-subpla.patch | text/x-diff | 50.3 KB |
| v14-0003-Common-infrastructure-to-MergeAppend-and-Append-node.patch | text/x-diff | 88.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alexander Lakhin | 2026-03-02 07:00:00 | Re: BUG: Former primary node might stuck when started as a standby |
| Previous Message | Chao Li | 2026-03-02 06:38:54 | Re: Reject ADD CONSTRAINT NOT NULL if name mismatches existing domain not-null constraint |