| From: | "Matheus Alcantara" <matheusssilv97(at)gmail(dot)com> |
|---|---|
| To: | "Alexander Pyhalov" <a(dot)pyhalov(at)postgrespro(dot)ru>, "Matheus Alcantara" <matheusssilv97(at)gmail(dot)com> |
| Cc: | "Alena Rybakina" <a(dot)rybakina(at)postgrespro(dot)ru>, "Pgsql Hackers" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Asynchronous MergeAppend |
| Date: | 2025-12-30 15:04:45 |
| Message-ID: | DFBN6K9K9IUG.23WZ7OBL8DRAQ@gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
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.
--
Matheus Alcantara
EDB: https://www.enterprisedb.com
| Attachment | Content-Type | Size |
|---|---|---|
| v12-0001-mark_async_capable-subpath-should-match-subplan.patch | text/plain | 2.4 KB |
| v12-0002-MergeAppend-should-support-Async-Foreign-Scan-su.patch | text/plain | 50.3 KB |
| v12-0003-Common-infrastructure-to-MergeAppend-and-Append-.patch | text/plain | 87.3 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Kirill Reshke | 2025-12-30 15:14:41 | Re: REASSIGN OWNED BY alters objects in other database. |
| Previous Message | Bryan Green | 2025-12-30 15:01:11 | Python Limited API for PL/Python on MSVC |