fix

Author: ivanmorozov333

ydb/core/tx/columnshard/engines/storage/granule/granule.cpp

@@ -145,8 +145,8 @@ void TGranuleMeta::UpsertPortionOnLoad(std::shared_ptr<TPortionInfo>&& portion)
         AFL_VERIFY(InsertedPortions.emplace(insertWriteId, portion).second);
         AFL_VERIFY(!Portions.contains(portion->GetPortionId()));
     } else {
-        auto portionId = portion.GetPortionId();
-        AFL_VERIFY(Portions.emplace(portionId, std::make_shared<TPortionInfo>(std::move(portion))).second);
+        auto portionId = portion->GetPortionId();
+        AFL_VERIFY(Portions.emplace(portionId, portion)).second);
     }
 }
 

ydb/core/tx/columnshard/engines/ut/ut_logs_engine.cpp

@@ -89,14 +89,19 @@ class TTestDbWrapper : public IDbWrapper {
     }
 
     virtual void WritePortion(const NOlap::TPortionInfo& portion) override {
-        Portions[portion.GetPortionId()] = portion.MakeCopy();
+        auto it = Portions.find(portion.GetPortionId());
+        if (it == Portions.end()) {
+            Portions.emplace(portion.GetPortionId(), portion.MakeCopy());
+        } else {
+            it->second = portion.MakeCopy();
+        }
     }
     virtual void ErasePortion(const NOlap::TPortionInfo& portion) override {
         AFL_VERIFY(Portions.erase(portion.GetPortionId()));
     }
     virtual bool LoadPortions(const std::function<void(NOlap::TPortionInfoConstructor&&, const NKikimrTxColumnShard::TIndexPortionMeta&)>& callback) override {
         for (auto&& i : Portions) {
-            callback(NOlap::TPortionInfoConstructor(i, false, true), i.GetMeta().SerializeToProto());
+            callback(NOlap::TPortionInfoConstructor(i.second, false, true), i.second.GetMeta().SerializeToProto());
         }
         return true;
     }