Hi, > > Also, should we define pg_get_injection_points in the injection_points > > extension maybe? > > As this is global for all libraries, that's not something I would add > there.
If I didn't miss anything, all SQL functions dealing with injection points are gathered in injection_points extension so IMO pg_get_injection_points() belongs there. It would be awkward to have it in the core considering the fact that injection points presumably should be disabled in release builds. Users will see a function in \df that does nothing. -- Best regards, Aleksander Alekseev