diff options
author | Peter Eisentraut <peter@eisentraut.org> | 2020-11-04 07:47:06 +0100 |
---|---|---|
committer | Peter Eisentraut <peter@eisentraut.org> | 2020-11-04 12:46:28 +0100 |
commit | 560564d3addcb6c54e13c1ca8bd11afafbb7ba11 (patch) | |
tree | 53ac268f527f647e95485ec6dde4c66a52d8026d /src/backend/utils/adt/arrayfuncs.c | |
parent | 37d2ff38031262a1778bc76a9c55fff7afbcf275 (diff) | |
download | postgresql-560564d3addcb6c54e13c1ca8bd11afafbb7ba11.tar.gz postgresql-560564d3addcb6c54e13c1ca8bd11afafbb7ba11.zip |
Enable hash partitioning of text arrays
hash_array_extended() needs to pass PG_GET_COLLATION() to the hash
function of the element type. Otherwise, the hash function of a
collation-aware data type such as text will error out, since the
introduction of nondeterministic collation made hash functions require
a collation, too.
The consequence of this is that before this change, hash partitioning
using an array over text in the partition key would not work.
Reviewed-by: Heikki Linnakangas <hlinnaka@iki.fi>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Reviewed-by: Michael Paquier <michael@paquier.xyz>
Discussion: https://www.postgresql.org/message-id/flat/32c1fdae-95c6-5dc6-058a-a90330a3b621%40enterprisedb.com
Diffstat (limited to 'src/backend/utils/adt/arrayfuncs.c')
-rw-r--r-- | src/backend/utils/adt/arrayfuncs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/adt/arrayfuncs.c b/src/backend/utils/adt/arrayfuncs.c index 392445ea032..a7ea7656c75 100644 --- a/src/backend/utils/adt/arrayfuncs.c +++ b/src/backend/utils/adt/arrayfuncs.c @@ -4071,7 +4071,7 @@ hash_array_extended(PG_FUNCTION_ARGS) typalign = typentry->typalign; InitFunctionCallInfoData(*locfcinfo, &typentry->hash_extended_proc_finfo, 2, - InvalidOid, NULL, NULL); + PG_GET_COLLATION(), NULL, NULL); /* Loop over source data */ nitems = ArrayGetNItems(ndims, dims); |