]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Revert refactoring of hex code to src/common/
authorMichael Paquier <michael@paquier.xyz>
Thu, 19 Aug 2021 00:20:19 +0000 (09:20 +0900)
committerMichael Paquier <michael@paquier.xyz>
Thu, 19 Aug 2021 00:20:19 +0000 (09:20 +0900)
commit1900c140554efdcaa94134705e9be7ce1437be9c
tree303a405b57767d4eb8c141603719c8357c2e7f34
parent61f9dae2ce7680d51035e5bb8668cf5227346eac
Revert refactoring of hex code to src/common/

This is a combined revert of the following commits:
c3826f8, a refactoring piece that moved the hex decoding code to
src/common/.  This code was cleaned up by aef8948, as it originally
included no overflow checks in the same way as the base64 routines in
src/common/ used by SCRAM, making it unsafe for its purpose.
aef8948, a more advanced refactoring of the hex encoding/decoding code
to src/common/ that added sanity checks on the result buffer for hex
decoding and encoding.  As reported by Hans Buschmann, those overflow
checks are expensive, and it is possible to see a performance drop in
the decoding/encoding of bytea or LOs the longer they are.  Simple SQLs
working on large bytea values show a clear difference in perf profile.
ccf4e27, a cleanup made possible by aef8948.

The reverts of all those commits bring back the performance of hex
decoding and encoding back to what it was in ~13.  Fow now and
post-beta3, this is the simplest option.

Reported-by: Hans Buschmann
Discussion: https://postgr.es/m/1629039545467.80333@nidsa.net
Backpatch-through: 14
src/backend/replication/backup_manifest.c
src/backend/utils/adt/encode.c
src/backend/utils/adt/varlena.c
src/common/Makefile
src/common/hex.c [deleted file]
src/include/common/hex.h [deleted file]
src/include/common/sha2.h
src/include/utils/builtins.h
src/tools/msvc/Mkvcbuild.pm