Adapt hashfn.c and hashutils.h for frontend use.
authorRobert Haas <rhaas@postgresql.org>
Mon, 24 Feb 2020 11:57:15 +0000 (17:27 +0530)
committerRobert Haas <rhaas@postgresql.org>
Mon, 24 Feb 2020 11:57:15 +0000 (17:27 +0530)
commita91e2fa94180f24dd68fb6c99136cda820e02089
tree0aac0e3930a043b621e5be2f82142d21c913b127
parent9341c783cc42ffae5860c86bdc713bd47d734ffd
Adapt hashfn.c and hashutils.h for frontend use.

hash_any() and its various variants are defined to return Datum,
which is a backend-only concept, but the underlying functions
actually want to return uint32 and uint64, and only return Datum
because it's convenient for callers who are using them to
implement a hash function for some SQL datatype.

However, changing these functions to return uint32 and uint64
seems like it might lead to programming errors or back-patching
difficulties, both because they are widely used and because
failure to use UInt{32,64}GetDatum() might not provoke a
compilation error. Instead, rename the existing functions as
well as changing the return type, and add static inline wrappers
for those callers that need the previous behavior.

Although this commit adapts hashutils.h and hashfn.c so that they
can be compiled as frontend code, it does not actually do
anything that would cause them to be so compiled. That is left
for another commit.

Patch by me, reviewed by Suraj Kharage and Mark Dilger.

Discussion: http://postgr.es/m/CA+TgmoaRiG4TXND8QuM6JXFRkM_1wL2ZNhzaUKsuec9-4yrkgw@mail.gmail.com
src/backend/utils/hash/hashfn.c
src/include/utils/hashutils.h