On Thu, Aug 15, 2024 at 4:00 AM Nathan Bossart <nathandboss...@gmail.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
>

Reply via email to