pgsql: Have pg_terminate/cancel_backend not ERROR on non-existent proce - Mailing list pgsql-committers

From Alvaro Herrera
Subject pgsql: Have pg_terminate/cancel_backend not ERROR on non-existent proce
Date
Msg-id E1THJd3-0004Sp-Di@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Have pg_terminate/cancel_backend not ERROR on non-existent processes

This worked fine for superusers, but not for ordinary users trying to
cancel their own processes.  Tweak the order the checks are done in so
that we correctly return SIGNAL_BACKEND_ERROR (which current callers
know to ignore without erroring out) so that an ordinary user can loop
through a resultset without fearing that a process might exit in the
middle of said looping -- causing the remaining processes to go
unsignalled.

Incidentally, the last in-core caller of IsBackendPid() is now gone.
However, the function is exported and must remain in place, because
there are plenty of callers in external modules.

Author: Josh Kupershmidt

Reviewed by Noah Misch

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/ae90ffada4cb27c9194797c43d864a86ce88d72c

Modified Files
--------------
src/backend/utils/adt/misc.c |   39 ++++++++++++++++-----------------------
1 files changed, 16 insertions(+), 23 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Run check_keywords.pl anytime gram.c is rebuilt.
Next
From: Alvaro Herrera
Date:
Subject: pgsql: Have pg_terminate/cancel_backend not ERROR on non-existent proce