thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-27 18:12:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Rework AggregateFunctionQuantile.h fix

Browse Source
This commit is contained in:
Jordi Villar 2023-03-27 13:23:12 +02:00
parent 4eafb1ae6b
commit b64cb10d20
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/AggregateFunctions/AggregateFunctionQuantile.h
View File

@ -125,8 +125,15 @@ public:
if constexpr (std::is_same_v<Data, QuantileTiming<Value>>) if constexpr (std::is_same_v<Data, QuantileTiming<Value>>)
{ {
/// QuantileTiming only supports unsigned integers. Too large values are also meaningless. /// QuantileTiming only supports unsigned integers. Too large values are also meaningless.
if (isNaN(value) || value > std::numeric_limits<Value>::max() || value < 0) #ifdef OS_DARWIN
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wimplicit-const-int-float-conversion"
#endif
if (isNaN(value) || value > std::numeric_limits<Int64>::max() || value < 0)
return; return;
#ifdef OS_DARWIN
#pragma clang diagnostic pop
#endif
} }
if constexpr (has_second_arg) if constexpr (has_second_arg)