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
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 |