Re: [Proposal] Adding callback support for custom statistics kinds

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Peter Eisentraut <peter(at)eisentraut(dot)org>
Cc: Sami Imseih <samimseih(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [Proposal] Adding callback support for custom statistics kinds
Date: 2026-07-13 03:38:34
Message-ID: alRduZzb_ceuc_aZ@paquier.xyz
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Jul 09, 2026 at 11:10:18AM +0200, Peter Eisentraut wrote:
> I suggest that unless we can come up with a well-defined, robust, and
> well-documented API for these, they should be moved back to be internal to
> pgstat.c, and test_custom_var_stats.c can define its own helper functions.
> Maybe in the future, when we have more of these, some good API will emerge.

I am not enough attached to these two APIs to keep them published if
one feels that it is weird to have them; we are talking about only a
few lines of code duplicated between core and the test module. I
still find the separation cleaner as done on HEAD, but well that's
just one opinion.

Sami, what's your take?
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message shveta malik 2026-07-13 04:05:30 Re: sequencesync worker race with REFRESH SEQUENCES
Previous Message Noah Misch 2026-07-13 03:35:36 Re: sequencesync worker race with REFRESH SEQUENCES