| From: | SAMEER KUMAR <sameer(dot)kasi200x(at)gmail(dot)com> |
|---|---|
| To: | Nathan Bossart <nathandbossart(at)gmail(dot)com> |
| Cc: | PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Adding clarification to description of IPC wait events XactGroupUpdate and ProcArrayGroupUpdate |
| Date: | 2024-08-15 03:25:25 |
| Message-ID: | CAGPeHmi1j9itUBp8iVzD6b1UqaxuV=N2vrOeKfgyka2O=T2EtQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Aug 15, 2024 at 4:00 AM Nathan Bossart <nathandbossart(at)gmail(dot)com>
wrote:
> On Wed, Aug 14, 2024 at 10:38:49PM +0800, SAMEER KUMAR wrote:
> > Yes, while reviewing the history of changes, I too noticed the same. The
> > documentation of older versions (v12 [1]) still has old descriptions.
>
> After reading the related threads and code, I'm inclined to agree that this
> is a mistake, or at least that the current wording is likely to mislead
> folks into thinking it has something to do with parallel query. I noticed
> that your patch changed a few things in the description, but IMHO we should
> keep the fix focused, i.e., just replace "end of a parallel operation" with
> "transaction end." I've attached a modified version of the patch with this
> change.
>
Thanks for the feedback Nathan.
I think it is important to indicate that the group leader is responsible
for clearing the transaction ID/transaction status of other backends
(including this one).
If you suggest that we keep it simple, I don't see any other issues with
your patch.
>
> --
> nathan
>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Joel Jacobson | 2024-08-15 04:19:36 | Re: [PATCH] Add get_bytes() and set_bytes() functions |
| Previous Message | Euler Taveira | 2024-08-15 03:16:40 | Re: Opinion poll: Sending an automated email to a thread when it gets added to the commitfest |