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

fix pvs check

Browse Source
This commit is contained in:
Anton Popov 2021-11-03 00:06:08 +03:00
parent 07d6013bc6
commit 44875ae79d
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Processors/Transforms/FillingTransform.cpp
View File

@ -45,6 +45,7 @@ static FillColumnDescription::StepFunction getStepFunction(
FOR_EACH_INTERVAL_KIND(DECLARE_CASE) FOR_EACH_INTERVAL_KIND(DECLARE_CASE)
#undef DECLARE_CASE #undef DECLARE_CASE
} }
__builtin_unreachable();
} }
static bool tryConvertFields(FillColumnDescription & descr, const DataTypePtr & type) static bool tryConvertFields(FillColumnDescription & descr, const DataTypePtr & type)
@ -102,7 +103,6 @@ static bool tryConvertFields(FillColumnDescription & descr, const DataTypePtr &
descr.step_func = getStepFunction<UInt32>(*descr.step_kind, get<Int64>(descr.fill_step), date_time->getTimeZone()); descr.step_func = getStepFunction<UInt32>(*descr.step_kind, get<Int64>(descr.fill_step), date_time->getTimeZone());
else if (const auto * date_time64 = checkAndGetDataType<DataTypeDateTime64>(type.get())) else if (const auto * date_time64 = checkAndGetDataType<DataTypeDateTime64>(type.get()))
{ {
const auto & time_zone = date_time64->getTimeZone();
const auto & step_dec = get<const DecimalField<Decimal64> &>(descr.fill_step); const auto & step_dec = get<const DecimalField<Decimal64> &>(descr.fill_step);
Int64 step = DecimalUtils::convertTo<Int64>(step_dec.getValue(), step_dec.getScale()); Int64 step = DecimalUtils::convertTo<Int64>(step_dec.getValue(), step_dec.getScale());
@ -110,7 +110,7 @@ static bool tryConvertFields(FillColumnDescription & descr, const DataTypePtr &
{ {
#define DECLARE_CASE(NAME) \ #define DECLARE_CASE(NAME) \
case IntervalKind::NAME: \ case IntervalKind::NAME: \
descr.step_func = [step, &time_zone](Field & field) \ descr.step_func = [step, &time_zone = date_time64->getTimeZone()](Field & field) \
{ \ { \
auto field_decimal = get<DecimalField<DateTime64>>(field); \ auto field_decimal = get<DecimalField<DateTime64>>(field); \
auto components = DecimalUtils::splitWithScaleMultiplier(field_decimal.getValue(), field_decimal.getScaleMultiplier()); \ auto components = DecimalUtils::splitWithScaleMultiplier(field_decimal.getValue(), field_decimal.getScaleMultiplier()); \