pgsql: Simplify abbreviated key hashing during cardinality estimation

From: John Naylor <john(dot)naylor(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Simplify abbreviated key hashing during cardinality estimation
Date: 2026-09-21 10:48:29
Message-ID: E1x8bZV-00000000dj7-2ODn@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Simplify abbreviated key hashing during cardinality estimation

In the wake of commit 2a600a93c, Datums are 64-bit values, so
we no longer need to fold the abbreviated key to 32 bits before
hashing. Instead, use murmurhash64(), which is simpler and avoids
a function call. Also replace hash_uint32() with murmurhash32()
in related places.

Author: Aleksander Alekseev <aleksander(at)tigerdata(dot)com>
Suggested-by: John Naylor <johncnaylorls(at)gmail(dot)com>
Reviewed-by: Aditya Gollamudi <adigollamudi(at)gmail(dot)com> (earlier version)
Reviewed-by: Michael Paquier <michael(at)paquier(dot)xyz> (earlier version)
Discussion: https://postgr.es/m/CAJ7c6TMPhDRQMmkUHPv8oOK97B1mR8NRS61DgjpdaZUPAwaeZQ%40mail.gmail.com

Branch
------
master

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

Modified Files
--------------
src/backend/utils/adt/bytea.c | 9 ++-------
src/backend/utils/adt/network.c | 9 ++-------
src/backend/utils/adt/numeric.c | 8 ++------
src/backend/utils/adt/uuid.c | 9 ++-------
src/backend/utils/adt/varlena.c | 9 ++-------
5 files changed, 10 insertions(+), 34 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2026-09-21 19:16:06 pgsql: Stamp 19beta4.
Previous Message Peter Eisentraut 2026-09-21 10:21:30 pgsql: Translation updates