| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
| Cc: | "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Shmem allocated wrong for custom cumulative stats |
| Date: | 2026-04-06 03:14:06 |
| Message-ID: | adMk_lWbnz3HDOA8@paquier.xyz |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Mon, Apr 06, 2026 at 11:36:21AM +0900, Michael Paquier wrote:
> On Mon, Apr 06, 2026 at 09:55:14AM +0900, Michael Paquier wrote:
>> Right, down to v18 where this has been introduced. That's my bug, so
>> I'd be OK to take care of it myself, if you are OK with that of
>> course.
>
> Note: something is wrong with -m32, for both patches. Digging into
> that..
And I have been puzzled for a few minutes here, trying to figure out
if this was something in v18 or something with the new shmem routines.
It is nothing of the kind: test_custom_stats has been underestimating
its shared_size, using PgStat_StatCustomFixedEntry instead of
PgStatShared_CustomFixedEntry. Interesting copy-pasto, HEAD-only,
second bug.
The attached is working correctly. The v18 flavor is slightly
simpler.
--
Michael
| Attachment | Content-Type | Size |
|---|---|---|
| v3-0001-Use-the-allocated-space-properly-for-custom--HEAD.patch | text/plain | 2.1 KB |
| v3-0001-Use-the-allocated-space-properly-for-custom-s-v18.patch | text/plain | 1.1 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Xuneng Zhou | 2026-04-06 03:26:59 | Re: Implement waiting for wal lsn replay: reloaded |
| Previous Message | Richard Guo | 2026-04-06 03:06:44 | Re: Eager aggregation, take 3 |