thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-28 12:40:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

trying to fix different types of *

Browse Source
This commit is contained in:
myrrc 2020-12-23 17:18:38 +03:00
parent fec0a59082
commit fcfe76d703
1 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/Functions/FunctionBinaryArithmetic.h
View File

@ -802,20 +802,26 @@ class FunctionBinaryArithmetic : public IFunction
if constexpr (IsDataTypeDecimal<RightDataType> && is_division)
return right.getScaleMultiplier(); // the division impl uses only the scale_a
else if constexpr (result_is_decimal)
return type.scaleFactorFor(left, is_multiply);
else if constexpr (left_is_decimal) // needed for e.g. multiply(Decimal, Float) to scale the arguments
// scale multiplier = 10^scale, so we need to get this value divided by 10^0 -> 2nd arg is 0 as
// the convertTo basically divides the given value on the second arg.
return 1 / DecimalUtils::convertTo<ResultType>(left.getScaleMultiplier(), 0);
// scale_a = 1 is result is decimal
return is_multiply ? 1 : type.scaleFactorFor(left, false);
else if constexpr (left_is_decimal)
{
const ResultType scale = DecimalUtils::convertTo<ResultType>(left.getScaleMultiplier(), 0);
// if const, the scale needs to be inverted
return !col_left_const ? scale : (1 / scale);
}
else
return 1; // the default value which won't cause any re-scale
}();
const ResultType scale_b = [&] {
if constexpr (result_is_decimal)
return type.scaleFactorFor(right, is_multiply || is_division);
return is_multiply ? 1 : type.scaleFactorFor(right, true);
else if constexpr (right_is_decimal)
return 1 / DecimalUtils::convertTo<ResultType>(right.getScaleMultiplier(), 0);
{
const ResultType scale = DecimalUtils::convertTo<ResultType>(right.getScaleMultiplier(), 0);
return !col_right_const ? scale : (1 / scale);
}
else
return 1;
}();