diff options
| author | Tomas Vondra | 2021-11-06 00:25:31 +0000 |
|---|---|---|
| committer | Tomas Vondra | 2021-11-06 00:50:44 +0000 |
| commit | d91353f4b21f10417d142e6ac17a0490adae867c (patch) | |
| tree | 8387c32185fab8c1c601a07c42aa17f9a3fb85cc /src/backend/access | |
| parent | f214960adde6028a39ba3014b1ab2b224faeefed (diff) | |
Fix handling of NaN values in BRIN minmax multi
When calculating distance between float4/float8 values, we need to be a
bit more careful about NaN values in order not to trigger assert. We
consider NaN values to be equal (distace 0.0) and in infinite distance
from all other values.
On builds without asserts, this issue is mostly harmless - the ranges
may be merged in less efficient order, but the index is still correct.
Per report from Andreas Seltenreich. Backpatch to 14, where this new
BRIN opclass was introduced.
Reported-by: Andreas Seltenreich
Discussion: https://postgr.es/m/87r1bw9ukm.fsf@credativ.de
Diffstat (limited to 'src/backend/access')
| -rw-r--r-- | src/backend/access/brin/brin_minmax_multi.c | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/src/backend/access/brin/brin_minmax_multi.c b/src/backend/access/brin/brin_minmax_multi.c index ee6c4795f1c..a85dfdfec45 100644 --- a/src/backend/access/brin/brin_minmax_multi.c +++ b/src/backend/access/brin/brin_minmax_multi.c @@ -73,6 +73,7 @@ #include "utils/builtins.h" #include "utils/date.h" #include "utils/datum.h" +#include "utils/float.h" #include "utils/inet.h" #include "utils/lsyscache.h" #include "utils/memutils.h" @@ -1872,6 +1873,14 @@ brin_minmax_multi_distance_float4(PG_FUNCTION_ARGS) float a1 = PG_GETARG_FLOAT4(0); float a2 = PG_GETARG_FLOAT4(1); + /* if both values are NaN, then we consider them the same */ + if (isnan(a1) && isnan(a2)) + PG_RETURN_FLOAT8(0.0); + + /* if one value is NaN, use infinite distance */ + if (isnan(a1) || isnan(a2)) + PG_RETURN_FLOAT8(get_float8_infinity()); + /* * We know the values are range boundaries, but the range may be collapsed * (i.e. single points), with equal values. @@ -1890,6 +1899,14 @@ brin_minmax_multi_distance_float8(PG_FUNCTION_ARGS) double a1 = PG_GETARG_FLOAT8(0); double a2 = PG_GETARG_FLOAT8(1); + /* if both values are NaN, then we consider them the same */ + if (isnan(a1) && isnan(a2)) + PG_RETURN_FLOAT8(0.0); + + /* if one value is NaN, use infinite distance */ + if (isnan(a1) || isnan(a2)) + PG_RETURN_FLOAT8(get_float8_infinity()); + /* * We know the values are range boundaries, but the range may be collapsed * (i.e. single points), with equal values. |