Re: Add pg_get_injection_points() for information of injection points

From: Kirill Reshke <reshkekirill(at)gmail(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Add pg_get_injection_points() for information of injection points
Date: 2025-04-14 12:26:14
Message-ID: CALdSSPh7jxiM0LOzBkEgB4xVqRXoM0DE8ac58kMOBPm+qijdvA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, 14 Apr 2025 at 05:36, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> Hi all,
>
> One thing that's been lacking in injection points is the possibility
> to look at the state of the injection points in shared memory through
> some SQL. This was on my tablets for some time, but I have not taken
> the time to do the actual legwork.
>
> The attached patch adds a SRF that returns a set of tuples made of the
> name, library and function for all the injection points attached to
> the system. This implementation relies on a new function added in
> injection_point.c, called InjectionPointList(), that retrieves a
> palloc()'d array of the injection points, hiding from the callers the
> internals of what this stuff does with the shmem array lookup.
>
> This is useful for monitoring or in tests, to make sure for example
> that nothing is left around at the end of a script. I have a
> different proposal planned for this area of the code, where this
> function would be good to have, but I am sending an independent patch
> as this stuff is useful on its own.
>
> The patch includes a couple of tests and some documentation.
>
> Thanks,
> --
> Michael

Hi! I noticed you do not bump catalog version here, so i was a little
confused with
```
reshke=# SELECT name, library, function FROM pg_get_injection_points();
ERROR: function pg_get_injection_points() does not exist
```
for a while

Also, should we define pg_get_injection_points in the injection_points
extension maybe?

--
Best regards,
Kirill Reshke

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2025-04-14 12:32:29 Re: Add pg_get_injection_points() for information of injection points
Previous Message Matheus Alcantara 2025-04-14 12:25:11 Re: New committer: Jacob Champion