injection_points: Fix incrementation of variable-numbered stats
authorMichael Paquier <michael@paquier.xyz>
Wed, 17 Sep 2025 01:15:13 +0000 (10:15 +0900)
committerMichael Paquier <michael@paquier.xyz>
Wed, 17 Sep 2025 01:15:13 +0000 (10:15 +0900)
commit39f67d9b554fcd6dfb6c4f02a4af757eecd1fecf
tree4e1a2db6cb37e8b3473e492b360b5499fa3016fb
parent158c48303e8ae0248d7d95b60b56dd67e7033c48
injection_points: Fix incrementation of variable-numbered stats

The pending entry was not used when incrementing its data, directly
manipulating the shared memory pointer, without even locking it.  This
could mean losing statistics under concurrent activity.  The flush
callback was a no-op.

This code serves as a base template for extensions for the custom
cumulative statistics, so let's be clean and use a pending entry for the
incrementations, whose data is then flushed to the corresponding entry
in the shared hashtable when all the stats are reported, in its own
flush callback.

Author: Sami Imseih <samimseih@gmail.com>
Discussion: https://postgr.es/m/CAA5RZ0v0U0yhPbY+bqChomkPbyUrRQ3rQXnZf_SB-svDiQOpgQ@mail.gmail.com
Backpatch-through: 18
src/test/modules/injection_points/injection_stats.c