Re: [PATCH] ltree hash functions - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCH] ltree hash functions
Date
Msg-id 755500.1711060183@sss.pgh.pa.us
Whole thread Raw
In response to Re: [PATCH] ltree hash functions  (jian he <jian.universality@gmail.com>)
List pgsql-hackers
jian he <jian.universality@gmail.com> writes:
> I also made the following changes:
> from

> uint64 levelHash = hash_any_extended((unsigned char *) al->name, al->len, seed);
> uint32 levelHash = hash_any((unsigned char *) al->name, al->len);

> to
> uint64 levelHash = DatumGetUInt64(hash_any_extended((unsigned char *)
> al->name, al->len, seed));
> uint32 levelHash = DatumGetUInt32(hash_any((unsigned char *) al->name,
> al->len));

Yeah, that'd fail on 32-bit machines.

Pushed v5 with some minor cosmetic tweaking.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: Adding comments to help understand psql hidden queries
Next
From: "David G. Johnston"
Date:
Subject: Re: documentation structure