pgsql: Convert uses of hash_string_pointer to fasthash equivalent - Mailing list pgsql-committers

From John Naylor
Subject pgsql: Convert uses of hash_string_pointer to fasthash equivalent
Date
Msg-id E1rsyUT-000l5f-1H@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Convert uses of hash_string_pointer to fasthash equivalent

Remove duplicate hash_string_pointer() function definitions by creating
a new inline function hash_string() for this purpose.

This has the added advantage of avoiding strlen() calls when doing hash
lookup. It's not clear how many of these are perfomance-sensitive
enough to benefit from that, but the simplification is worth it on
its own.

Reviewed by Jeff Davis

Discussion: https://postgr.es/m/CANWCAZbg_XeSeY0a_PqWmWqeRATvzTzUNYRLeT%2Bbzs%2BYQdC92g%40mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/f956ecd0353b2960f8322b2211142113fe2b6f67

Modified Files
--------------
src/bin/pg_combinebackup/load_manifest.c  | 16 ++--------------
src/bin/pg_dump/pg_dumpall.c              | 17 ++---------------
src/bin/pg_rewind/filemap.c               | 17 ++---------------
src/bin/pg_verifybackup/pg_verifybackup.c | 16 ++--------------
src/include/common/hashfn_unstable.h      | 20 ++++++++++++++++++++
5 files changed, 28 insertions(+), 58 deletions(-)


pgsql-committers by date:

Previous
From: Thomas Munro
Date:
Subject: pgsql: Improve read_stream.c's fast path.
Next
From: Thomas Munro
Date:
Subject: pgsql: Increase default vacuum_buffer_usage_limit to 2MB.