Schemeshard stats histrogram avoid TSerializedCellVec serialization

1 files changed, 8 insertions, 4 deletions

diff --git a/ydb/core/tx/schemeshard/schemeshard__table_stats_histogram.cpp b/ydb/core/tx/schemeshard/schemeshard__table_stats_histogram.cpp
index 071f776fe7f..d736d43282d 100644
--- a/ydb/core/tx/schemeshard/schemeshard__table_stats_histogram.cpp
+++ b/ydb/core/tx/schemeshard/schemeshard__table_stats_histogram.cpp
@@ -85,7 +85,7 @@ TSerializedCellVec ChooseSplitKeyByHistogram(const NKikimrTableStats::THistogram
         break;
     }
 
-    return TSerializedCellVec(TSerializedCellVec::Serialize(splitKey));
+    return TSerializedCellVec(splitKey);
 }
 
 TSerializedCellVec DoFindSplitKey(const TVector<std::pair<TSerializedCellVec, ui64>>& keysHist,
@@ -121,13 +121,17 @@ TSerializedCellVec DoFindSplitKey(const TVector<std::pair<TSerializedCellVec, ui
     splitKey.resize(prefixSize);
     splitKey.resize(keyColumnTypes.size());
 
-    return TSerializedCellVec(TSerializedCellVec::Serialize(splitKey));
+
+    return TSerializedCellVec(splitKey);
 }
 
 TSerializedCellVec ChooseSplitKeyByKeySample(const NKikimrTableStats::THistogram& keySample, const TConstArrayRef<NScheme::TTypeInfo>& keyColumnTypes) {
     TVector<std::pair<TSerializedCellVec, ui64>> keysHist;
-    for (const auto& s : keySample.GetBuckets()) {
-        keysHist.emplace_back(std::make_pair(TSerializedCellVec(s.GetKey()), s.GetValue()));
+    const auto & buckets = keySample.GetBuckets();
+    keysHist.reserve(buckets.size());
+
+    for (const auto& bucket : buckets) {
+        keysHist.emplace_back(std::make_pair(TSerializedCellVec(bucket.GetKey()), bucket.GetValue()));
     }
 
     auto fnCmp = [&keyColumnTypes] (const auto& key1, const auto& key2) {