Re: question about pending updates in pgstat_report_inj - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: question about pending updates in pgstat_report_inj
Date
Msg-id aMjLpiX6reRycFVm@paquier.xyz
Whole thread Raw
In response to Re: question about pending updates in pgstat_report_inj  (Sami Imseih <samimseih@gmail.com>)
List pgsql-hackers
On Mon, Sep 15, 2025 at 05:33:45PM -0500, Sami Imseih wrote:
> I think it's better to use ->pending here, since this is referenced
> as an example and most real-world cases will likely want to use
> ->pending for performance reasons.

Yes, it should use the pending entry.  b757abefc041 did not get that
completely right.

The purpose of this code is also to serve as a template, so better
that it does the correct thing.

How about renaming "statent" to "pending" in pgstat_report_inj(), as
well?  That would be a bit more consistent with the subscription stat
case, at least.
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: BUG #18959: Name collisions of expression indexes during parallel Index creations on a pratitioned table.
Next
From: Kyotaro Horiguchi
Date:
Subject: Reword messages using "as" instead of "because"