Re: [PATCHES] updated hash functions for postgresql v1 - Mailing list pgsql-hackers

From Jeff Davis
Subject Re: [PATCHES] updated hash functions for postgresql v1
Date
Msg-id 1231613775.25019.110.camel@jdavis
Whole thread Raw
In response to Re: [PATCHES] updated hash functions for postgresql v1  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [PATCHES] updated hash functions for postgresql v1  (Kenneth Marshall <ktm@rice.edu>)
List pgsql-hackers
On Sat, 2009-01-10 at 13:36 -0500, Tom Lane wrote:
> Jeff Davis <pgsql@j-davis.com> writes:
> > I ran 5 times on both old and new code, eliminating the top and bottom
> > and taking the average of the remaining 3, and I got a 6.9% performance
> > improvement with the new code.
> 
> The question that has been carefully evaded throughout the discussion
> of this patch is whether the randomness of the hash result is decreased,
> and if so what is that likely to cost us in performance of the various
> hash-dependent algorithms.  I would still like to have an answer to that
> before we make a change to gain marginal performance improvement in
> the hash function itself (which is already shown to be barely measurable
> in the total context of a hash-dependent operation...)
> 

In:
http://archives.postgresql.org/message-id/20081104202655.GP18362@it.is.rice.edu

Ken showed that the number of hash collisions is the same in the old and
the new code for his test data. Not a direct measurement of randomness,
but it's some kind of indication.

I'm not an authority on either hash functions or statistics, so I'll
have to defer to Ken or someone else to actually prove that the
randomness is maintained.

Regards,Jeff Davis



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [SPAM] Re: posix_fadvise v22
Next
From: Gregory Stark
Date:
Subject: Re: [PATCHES] updated hash functions for postgresql v1