Remove excess explicit upcast and temporary UnboxedValuePod creation

commit_hash:cca2568fdd279c80e1af6d3c056c7bed92f690ca

Author: igormunkin

yql/essentials/minikql/comp_nodes/mkql_condense.cpp

@@ -30,7 +30,7 @@ class TCondenseFlowWrapper : public TStatefulFlowCodegeneratorNode<TCondenseFlow
 
     NUdf::TUnboxedValuePod DoCalculate(NUdf::TUnboxedValue& state, TComputationContext& ctx) const {
         if (state.IsFinish()) {
-            return static_cast<const NUdf::TUnboxedValuePod&>(state);
+            return state;
         }
 
         if (state.IsInvalid()) {

yql/essentials/minikql/comp_nodes/mkql_condense1.cpp

@@ -30,7 +30,7 @@ class TCondense1FlowWrapper : public TStatefulFlowCodegeneratorNode<TCondense1Fl
 
     NUdf::TUnboxedValuePod DoCalculate(NUdf::TUnboxedValue& state, TComputationContext& ctx) const {
         if (state.IsFinish()) {
-            return static_cast<const NUdf::TUnboxedValuePod&>(state);
+            return state;
         } else if (state.HasValue()) {
             if constexpr (UseCtx) {
                 CleanupCurrentContext();

yql/essentials/minikql/comp_nodes/mkql_flatmap.cpp

@@ -219,7 +219,7 @@ using TBaseComputation = TStatefulFlowCodegeneratorNode<TListFlatMapFlowWrapper>
         }
 
         if (state.IsFinish()) {
-            return NUdf::TUnboxedValuePod::MakeFinish();
+            return state;
         }
 
         while (true) {

yql/essentials/minikql/comp_nodes/mkql_multimap.cpp

@@ -21,8 +21,9 @@ class TFlowMultiMapWrapper : public TStatefulFlowCodegeneratorNode<TFlowMultiMap
     {}
 
     NUdf::TUnboxedValuePod DoCalculate(NUdf::TUnboxedValue& state, TComputationContext& ctx) const {
-        if (state.IsFinish())
-            return NUdf::TUnboxedValuePod::MakeFinish();
+        if (state.IsFinish()) {
+            return state;
+        }
 
         const auto pos = state.IsInvalid() ? 0ULL : state.Get<ui64>();
         if (!pos) {
@@ -412,8 +413,9 @@ using TBaseComputation = TStatefulFlowCodegeneratorNode<TNarrowMultiMapWrapper>;
     {}
 
     NUdf::TUnboxedValuePod DoCalculate(NUdf::TUnboxedValue& state, TComputationContext& ctx) const {
-        if (state.IsFinish())
-            return NUdf::TUnboxedValuePod::MakeFinish();
+        if (state.IsFinish()) {
+            return state;
+        }
 
         const auto pos = state.IsInvalid() ? 0ULL : state.Get<ui64>();
         if (!pos) {

yql/essentials/minikql/comp_nodes/mkql_squeeze_to_list.cpp

@@ -47,7 +47,7 @@ class TSqueezeToListWrapper: public TStatefulFlowCodegeneratorNode<TSqueezeToLis
 
     NUdf::TUnboxedValuePod DoCalculate(NUdf::TUnboxedValue& state, TComputationContext& ctx) const {
         if (state.IsFinish()) {
-            return NUdf::TUnboxedValuePod::MakeFinish();
+            return state;
         } else if (state.IsInvalid()) {
             MakeState(ctx, Limit->GetValue(ctx).GetOrDefault(std::numeric_limits<ui64>::max()), state);
         }

yt/yql/providers/yt/comp_nodes/dq/dq_yt_writer.cpp

@@ -87,7 +87,7 @@ class TYtDqWideWriteWrapper final : public TStatefulFlowCodegeneratorNode<TYtDqW
 
     NUdf::TUnboxedValuePod DoCalculate(NUdf::TUnboxedValue& state, TComputationContext& ctx) const {
         if (state.IsFinish()) {
-            return NUdf::TUnboxedValuePod::MakeFinish();
+            return state;
         } else if (state.IsInvalid())
             MakeState(ctx, state);
 
@@ -100,7 +100,7 @@ class TYtDqWideWriteWrapper final : public TStatefulFlowCodegeneratorNode<TYtDqW
             case EFetchResult::Finish:
                 ptr->Finish();
                 state = NUdf::TUnboxedValuePod::MakeFinish();
-                return NUdf::TUnboxedValuePod::MakeFinish();
+                return state;
         }
     }
 #ifndef MKQL_DISABLE_CODEGEN