| From: | John Naylor <john(dot)naylor(at)postgresql(dot)org> |
|---|---|
| To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | pgsql: Convert uses of hash_string_pointer to fasthash equivalent |
| Date: | 2024-04-06 05:21:21 |
| Message-ID: | E1rsyUT-000l5f-1H@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | 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(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Thomas Munro | 2024-04-06 10:12:50 | pgsql: Increase default vacuum_buffer_usage_limit to 2MB. |
| Previous Message | Thomas Munro | 2024-04-06 05:18:57 | pgsql: Improve read_stream.c's fast path. |