Re: Printing backtrace of postgres processes
vignesh C <vignesh21@gmail.com>
From: vignesh C <vignesh21@gmail.com>
To: Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>
Cc: Robert Haas <robertmhaas@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, Andres Freund <andres@anarazel.de>,
Craig Ringer <craig.ringer@enterprisedb.com>, Peter Eisentraut <peter.eisentraut@enterprisedb.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2021-07-21T17:26:04Z
Lists: pgsql-hackers
On Wed, Jul 21, 2021 at 3:52 PM Bharath Rupireddy < bharath.rupireddyforpostgres@gmail.com> wrote: > > On Tue, Jul 13, 2021 at 9:03 PM vignesh C <vignesh21@gmail.com> wrote: > > > > On Wed, May 12, 2021 at 2:27 AM Robert Haas <robertmhaas@gmail.com> wrote: > > > > > > Maybe we should have a role that is specifically for server debugging > > > type things. This kind of overlaps with Mark Dilger's proposal to try > > > to allow SET for security-sensitive GUCs to be delegated via > > > predefined roles. The exact way to divide that up is open to question, > > > but it wouldn't seem crazy to me if the same role controlled the > > > ability to do this plus the ability to set the GUCs > > > backtrace_functions, debug_invalidate_system_caches_always, > > > wal_consistency_checking, and maybe a few other things. > > > > +1 for the idea of having a new role for this. Currently I have > > implemented this feature to be supported only for the superuser. If we > > are ok with having a new role to handle debugging features, I will > > make a 002 patch to handle this. > > I see that there are a good number of user functions that are > accessible only by superuser (I searched for "if (!superuser())" in > the code base). I agree with the intention to not overload the > superuser anymore and have a few other special roles to delegate the > existing superuser-only functions to them. In that case, are we going > to revisit and reassign all the existing superuser-only functions? As Robert pointed out, this idea is based on Mark Dilger's proposal. Mark Dilger is already handling many of them at [1]. I'm proposing this patch only for server debugging functionalities based on Robert's suggestions at [2]. [1] - https://commitfest.postgresql.org/33/3223/ [2] - https://www.postgresql.org/message-id/CA%2BTgmoZz%3DK1bQRp0Ug%3D6uMGFWg-6kaxdHe6VSWaxq0U-YkppYQ%40mail.gmail.com Regards, Vignesh
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