pgsql: Avoid searching for callback functions in CallSyscacheCallbacks(

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Avoid searching for callback functions in CallSyscacheCallbacks(
Date: 2017-05-12 23:05:40
Message-ID: E1d9Jcy-0000aS-0E@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Avoid searching for callback functions in CallSyscacheCallbacks().

We have now grown enough registerable syscache-invalidation callback
functions that the original assumption that there would be few of them
is causing performance problems. In particular, let's fix things so that
CallSyscacheCallbacks doesn't have to search the whole array to find
which callback(s) to invoke for a given cache ID. Preserve the original
behavior that callbacks are called in order of registration, just in
case there's someplace that depends on that (which I doubt).

In support of this, export the number of syscaches from syscache.h.
People could have found that out anyway from the enum, but adding a
#define makes that much safer.

This provides a useful additional speedup in Mathieu Fenniak's
logical-decoding test case, although we're reaching the point of
diminishing returns there. I think any further improvement will have
to come from reducing the number of cache invalidations that are
triggered in the first place. Still, we can hope that this change
gives some incremental benefit for all invalidation scenarios.

Back-patch to 9.4 where logical decoding was introduced.

Discussion: https://postgr.es/m/CAHoiPjzea6N0zuCi=+f9v_j94nfsy6y8SU7-=bp4=7qw6_i=Rg@mail.gmail.com

Branch
------
REL9_4_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/32900700f4f5f373fb5ea9844b979a17bf6570a6

Modified Files
--------------
src/backend/utils/cache/inval.c | 39 ++++++++++++++++++++++++++++++++++----
src/backend/utils/cache/syscache.c | 5 +++--
src/include/utils/syscache.h | 2 ++
3 files changed, 40 insertions(+), 6 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2017-05-13 04:17:18 pgsql: Complete tab completion for DROP STATISTICS
Previous Message Tom Lane 2017-05-12 23:05:39 pgsql: Avoid searching for callback functions in CallSyscacheCallbacks(