Re: Pluggable cumulative statistics - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Pluggable cumulative statistics
Date
Msg-id ZoznS4q6W4XS4nua@paquier.xyz
Whole thread Raw
In response to Re: Pluggable cumulative statistics  (Bertrand Drouvot <bertranddrouvot.pg@gmail.com>)
List pgsql-hackers
On Tue, Jul 09, 2024 at 05:23:03AM +0000, Bertrand Drouvot wrote:
> I gave a second thought on it, and I think that this is the "data" part that lead
> to the confusion (as too generic), what about?
>
> shared_data_len -> shared_stats_len
> shared_data_off -> shared_stats_off
>
> That looks ok to me even in the snapshot context (shared is fine after all
> because that's where the stats come from).

I'd tend to prefer the original suggestion because of the snapshot
context, actually, as the fixed-numbered stats in a snapshot are a
copy of what's in shmem, and that's not shared at all.

The rename is not the most important part, still if others have an
opinion, feel free.
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Dean Rasheed
Date:
Subject: Re: Incorrect results from numeric round() and trunc()
Next
From: Michael Paquier
Date:
Subject: Re: Test to dump and restore objects left behind by regression