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:38 +0100 |
| commit | 82d4a2a7d63e79f6a6724f366cfaa4beed6b8326 (patch) | |
| tree | f69d6de6fe4c8bb1edccf22dbe86857802219ebe /src/backend/utils/adt/arrayfuncs.c | |
| parent | 7d39586a59c6f5816a068bd38e1e4887d4c984ff (diff) | |
| download | postgresql-82d4a2a7d63e79f6a6724f366cfaa4beed6b8326.tar.gz postgresql-82d4a2a7d63e79f6a6724f366cfaa4beed6b8326.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 800107d4e72..43354cc31e6 100644 --- a/src/backend/utils/adt/arrayfuncs.c +++ b/src/backend/utils/adt/arrayfuncs.c @@ -4050,7 +4050,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); |