Re: Remove traces of long in dynahash.c - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Remove traces of long in dynahash.c
Date
Msg-id aMENtE9cqL8nAW5o@paquier.xyz
Whole thread Raw
In response to Re: Remove traces of long in dynahash.c  (Dean Rasheed <dean.a.rasheed@gmail.com>)
List pgsql-hackers
On Tue, Sep 09, 2025 at 10:28:13AM +0100, Dean Rasheed wrote:
> So I think there's no point in adding that cap, or any additional
> checks in ExecChooseHashTableSize().

You are right that this hardcoded limit introduced in the previous
patch was useless.  So I have removed that, and applied the result.
Thanks for the suggestion.

Regarding removing the next power functions in dynahash.c, I am not
sure if it is worth bothering much.  Now that dynahash.h is removed,
all the code duplication that was in the backend, without the
hardcoded thresholds, is removed.
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Chao Li
Date:
Subject: Re: Mark ItemPointer arguments as const thoughoutly
Next
From: David Rowley
Date:
Subject: Re: Query Performance Degradation Due to Partition Scan Order – PostgreSQL v17.6