On Wed, Sep 23, 2026 at 3:50 PM Zsolt Parragi <[email protected]> wrote: > > Hello > > - h = DatumGetUInt32(hash_uint32(k->dboid)); > - h ^= DatumGetUInt32(hash_any((const unsigned char *) k->channel, > - > strnlen(k->channel, NAMEDATALEN))); > + h = murmurhash32(k->dboid); > + h ^= hash_bytes((const unsigned char *) k->channel, > + strnlen(k->channel, NAMEDATALEN)); > > I see that this change was discussed previously, but shouldn't it be > at least mentioned in the commit message?
I'm thinking now to leave the change to hash_uint32 out now, since there are places elsewhere that do round-trip casting via hash_uint32, and that could be material a separate patch, leaving this patch only touching hash_any. I'll make the change my self and commit soon. -- John Naylor Amazon Web Services
