thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-21 23:21:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

replaced ternary if

Browse Source
This commit is contained in:
myrrc 2020-12-18 16:35:37 +03:00
parent 84013d253d
commit 15f0915ad3
1 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/Functions/FunctionBinaryArithmetic.h
View File

@ -1025,21 +1025,27 @@ public:
else if constexpr (result_is_decimal)
left_scale = type.scaleFactorFor(left, is_multiply); //both are decimal, so multiply on 1 / scale
else if constexpr (left_is_decimal)
{
// While multiplying constants, we need the Decimal scales as the conversion function from
// Decimal utils takes it. When one of the columns is non-const, we need the scale multiplier
// instead as the OpImpl::constantConstant function uses the multiplier.
left_scale = both_columns_are_const
? left.getScale()
: left.template getScaleMultiplier<FieldType>();
if (both_columns_are_const)
left_scale = left.getScale();
else
left_scale = left.getScaleMultiplier();
}
else
left_scale = 0.0; //won't be used, just to silence the warning
if constexpr (result_is_decimal)
right_scale = type.scaleFactorFor(right, is_multiply || is_division);
else if constexpr (right_is_decimal)
right_scale = both_columns_are_const
? right.getScale()
: right.template getScaleMultiplier<FieldType>();
{
if (both_columns_are_const)
right_scale = right.getScale();
else
right_scale = right.getScaleMultiplier();
}
else
right_scale = 0.0; //same