Re: [PATCH] Refactor *_abbrev_convert() functions - Mailing list pgsql-hackers

From Zsolt Parragi
Subject Re: [PATCH] Refactor *_abbrev_convert() functions
Date
Msg-id CAN4CZFNuB8=cQj6K3oqTdcBh_gFxU8ZpZwS8JHv9Xs+O61XabA@mail.gmail.com
Whole thread
In response to Re: [PATCH] Refactor *_abbrev_convert() functions  (Aleksander Alekseev <aleksander@tigerdata.com>)
Responses Re: [PATCH] Refactor *_abbrev_convert() functions
List pgsql-hackers
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?



pgsql-hackers by date:

Previous
From: Amit Langote
Date:
Subject: Re: Revert RI fast-path batching from REL_19_STABLE
Next
From: shveta malik
Date:
Subject: Re: Crashes on a partition whose concurrent detach never finished