| From: | vignesh C <vignesh21(at)gmail(dot)com> |
|---|---|
| To: | Dilip Kumar <dilipbalaut(at)gmail(dot)com> |
| Cc: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Incorrect CONTEXT reported for errors from parallel apply worker in logical replication |
| Date: | 2026-11-09 17:35:22 |
| Message-ID: | CALDaNm1cBVsv5nXA731vv6M9MoBXpUwjXPLhTdx+yXfqbNp=fg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, 6 Nov 2026 at 11:34, Dilip Kumar <dilipbalaut(at)gmail(dot)com> wrote:
>
> Yeah this is better. I just have a suggestion that might sound better,
> but feel free to ignore it.
>
> errmsg("logical replication parallel apply worker exited due to error"),
> errdetail("The leader apply worker will now exit and terminate any
> other parallel apply workers.")
Thanks for the comments, the updated v9 version patch has the changes
to fix the same. The other comment regarding stale comments from [1]
also is addressed in this version.
[1] - https://www.postgresql.org/message-id/CAFiTN-sKca5sRc1xVL9ghdYT%3DhKtzoNoKMVwbrE1G55J0pdC9A%40mail.gmail.com
Regards,
Vignesh
| Attachment | Content-Type | Size |
|---|---|---|
| v9-0001-Fix-the-error-context-of-the-leader-apply-worker-.patch | application/octet-stream | 3.1 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Hannu Krosing | 2026-11-09 17:54:57 | Re: ZSTD TOAST compression, and an extensible compression method encoding |
| Previous Message | Greg Burd | 2026-11-09 16:43:09 | Re: use radix tree for bitmap heap scan |