From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Centralize logic for skipping useless ereport/elog calls. |
Date: | 2020-11-24 00:11:10 |
Message-ID: | E1khLvK-0006aP-B1@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Centralize logic for skipping useless ereport/elog calls.
While ereport() and elog() themselves are quite cheap when the
error message level is too low to be printed, some places need to do
substantial work before they can call those macros at all. To allow
optimizing away such setup work when nothing is to be printed, make
elog.c export a new function message_level_is_interesting(elevel)
that reports whether ereport/elog will do anything. Make use of that
in various places that had ad-hoc direct tests of log_min_messages etc.
Also teach ProcSleep to use it to avoid some work. (There may well
be other places that could usefully use this; I didn't search hard.)
Within elog.c, refactor a little bit to avoid having duplicate copies
of the policy-setting logic. When that code was written, we weren't
relying on the availability of inline functions; so it had some
duplications in the name of efficiency, which I got rid of.
Alvaro Herrera and Tom Lane
Discussion: https://postgr.es/m/129515.1606166429@sss.pgh.pa.us
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/789b938bf2b8e38d0894261eae6bc84bbbb4714e
Modified Files
--------------
src/backend/access/transam/xact.c | 3 +-
src/backend/access/transam/xlogutils.c | 6 +-
src/backend/catalog/dependency.c | 8 +-
src/backend/replication/walreceiver.c | 2 +-
src/backend/replication/walsender.c | 4 +-
src/backend/storage/lmgr/proc.c | 40 +++++----
src/backend/utils/error/elog.c | 153 ++++++++++++++++++++-------------
src/include/utils/elog.h | 4 +-
8 files changed, 126 insertions(+), 94 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2020-11-24 03:48:18 | pgsql: Use macros instead of hardcoded offsets for LWLock initializatio |
Previous Message | David Rowley | 2020-11-24 00:10:17 | pgsql: Fix unportable usage of __has_attribute |