diff options
| author | pg <pg@yandex-team.com> | 2023-08-20 22:48:39 +0300 |
|---|---|---|
| committer | pg <pg@yandex-team.com> | 2023-08-20 23:17:28 +0300 |
| commit | d619e9fffe040fe2f8b4940b1a72cc1757538c8c (patch) | |
| tree | ff8fd3865f285f81198e72b0a3ba7f3ad2bb754d | |
| parent | e471c138ece2f1c20445bdf591efa73e80192f88 (diff) | |
| download | ydb-d619e9fffe040fe2f8b4940b1a72cc1757538c8c.tar.gz | |
| -rw-r--r-- | ydb/library/yql/core/type_ann/type_ann_join.cpp | 2 | ||||
| -rw-r--r-- | ydb/library/yql/core/type_ann/type_ann_list.cpp | 12 |
2 files changed, 7 insertions, 7 deletions
diff --git a/ydb/library/yql/core/type_ann/type_ann_join.cpp b/ydb/library/yql/core/type_ann/type_ann_join.cpp index 2008d37def..8879712c62 100644 --- a/ydb/library/yql/core/type_ann/type_ann_join.cpp +++ b/ydb/library/yql/core/type_ann/type_ann_join.cpp @@ -658,7 +658,7 @@ namespace NTypeAnnImpl { return IGraphTransformer::TStatus::Error; } - if (const auto status = NormalizeTupleOfAtoms(input, 7U, output, ctx.Expr); IGraphTransformer::TStatus::Ok != status) { + if (const auto status = NormalizeTupleOfAtoms(input, 7U, output, ctx.Expr); status != IGraphTransformer::TStatus::Ok) { return status; } diff --git a/ydb/library/yql/core/type_ann/type_ann_list.cpp b/ydb/library/yql/core/type_ann/type_ann_list.cpp index 549cfffc18..6e6eee3a13 100644 --- a/ydb/library/yql/core/type_ann/type_ann_list.cpp +++ b/ydb/library/yql/core/type_ann/type_ann_list.cpp @@ -2126,15 +2126,15 @@ namespace { if (stepType && IsDataTypeFloat(stepItemType->GetSlot())) { commonType = ((beginIsOpt || endIsOpt) && !stepIsOpt) ? ctx.Expr.MakeType<TOptionalExprType>(stepType) : stepType; - if (const auto status = TrySilentConvertTo(input->ChildRef(0U), *commonType, ctx.Expr); IGraphTransformer::TStatus::Ok != status) { - if (IGraphTransformer::TStatus::Error == status) { + if (const auto status = TrySilentConvertTo(input->ChildRef(0U), *commonType, ctx.Expr); status != IGraphTransformer::TStatus::Ok) { + if (status == IGraphTransformer::TStatus::Error) { ctx.Expr.AddError(TIssue(ctx.Expr.GetPosition(input->Child(0U)->Pos()), TStringBuilder() << "Impossible silent convert bound of type " << *input->Child(0U)->GetTypeAnn() << " into " << *commonType)); } return status; } - if (const auto status = TrySilentConvertTo(input->ChildRef(1U), *commonType, ctx.Expr); IGraphTransformer::TStatus::Ok != status) { - if (IGraphTransformer::TStatus::Error == status) { + if (const auto status = TrySilentConvertTo(input->ChildRef(1U), *commonType, ctx.Expr); status != IGraphTransformer::TStatus::Ok) { + if (status == IGraphTransformer::TStatus::Error) { ctx.Expr.AddError(TIssue(ctx.Expr.GetPosition(input->Child(1U)->Pos()), TStringBuilder() << "Impossible silent convert bound of type " << *input->Child(1U)->GetTypeAnn() << " into " << *commonType)); } @@ -2164,9 +2164,9 @@ namespace { if (const auto stepSlot = IsDataTypeDateOrTzDateOrInterval(slot) ? EDataSlot::Interval : MakeSigned(slot); stepItemType) { if (const auto requredStepType = slot == stepSlot ? commonItemType : ctx.Expr.MakeType<TDataExprType>(stepSlot); !IsSameAnnotation(*stepItemType, *requredStepType)) { - if (const auto status = TrySilentConvertTo(input->ChildRef(2U), *requredStepType, ctx.Expr); IGraphTransformer::TStatus::Repeat == status) + if (const auto status = TrySilentConvertTo(input->ChildRef(2U), *requredStepType, ctx.Expr); status == IGraphTransformer::TStatus::Repeat) return status; - else if (IGraphTransformer::TStatus::Error == status && !EnsureSpecificDataType(input->Tail().Pos(), *stepItemType, stepSlot, ctx.Expr)) + else if (status == IGraphTransformer::TStatus::Error && !EnsureSpecificDataType(input->Tail().Pos(), *stepItemType, stepSlot, ctx.Expr)) return status; } } else { |