Re: Printing backtrace of postgres processes
Andres Freund <andres@anarazel.de>
From: Andres Freund <andres@anarazel.de>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Robert Haas <robertmhaas@gmail.com>, vignesh C <vignesh21@gmail.com>, Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>, Craig Ringer <craig.ringer@enterprisedb.com>, Peter Eisentraut <peter.eisentraut@enterprisedb.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-05-06T19:19:05Z
Lists: pgsql-hackers
Hi, On 2021-05-06 14:56:09 -0400, Tom Lane wrote: > Robert Haas <robertmhaas@gmail.com> writes: > > On Wed, Feb 3, 2021 at 2:30 AM Tom Lane <tgl@sss.pgh.pa.us> wrote: > >> TBH, I'm leaning to the position that this should be superuser > >> only. > > > I agree that ordinary users shouldn't be able to trigger it, but I > > think it should be restricted to some predefined role, new or > > existing, rather than to superuser. I see no reason not to let > > individual users decide what risks they want to take. > > If we think it's worth having a predefined role for, OK. However, > I don't like the future I see us heading towards where there are > hundreds of random predefined roles. Is there an existing role > that it'd be reasonable to attach this ability to? It does seem like it'd be good to group it in with something else. There's nothing fitting 100% though. postgres[1475723][1]=# SELECT rolname FROM pg_roles WHERE rolname LIKE 'pg_%' ORDER BY rolname; ┌───────────────────────────┐ │ rolname │ ├───────────────────────────┤ │ pg_database_owner │ │ pg_execute_server_program │ │ pg_monitor │ │ pg_read_all_data │ │ pg_read_all_settings │ │ pg_read_all_stats │ │ pg_read_server_files │ │ pg_signal_backend │ │ pg_stat_scan_tables │ │ pg_write_all_data │ │ pg_write_server_files │ └───────────────────────────┘ (11 rows) We could fit it into pg_monitor, but it's probably a bit more impactful than most things in there? But I'd go for it anyway, I think. Greetings, Andres Freund
Commits
-
Perform apply of large transactions by parallel workers.
- 216a784829c2 16.0 cited
-
Enhance pg_log_backend_memory_contexts() for auxiliary processes.
- 790fbda90209 15.0 cited
-
Allow GRANT on pg_log_backend_memory_contexts().
- f0b051e322d5 15.0 cited
-
Move Perl test modules to a better namespace
- b3b4d8e68ae8 15.0 cited
-
Unify PostgresNode's new() and get_new_node() methods
- 201a76183e20 15.0 cited
-
Add backtrace support for error reporting
- 71a8a4f6e365 13.0 cited