HPCC-31003 Move instead of copying objects to improve performance

Signed-off-by: Shamser Ahmed <[email protected]>
hpcc-systems · Dec 20, 2024 · 085868d · 085868d
1 parent 70ccefc
commit 085868d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/common/workunit/workunit.cpp b/common/workunit/workunit.cpp
@@ -2708,7 +2708,7 @@ void StatisticsAggregator::recordStats(IStatisticCollection * sourceStats, unsig
     verifyex(graphScopeId.setScopeText(graphName));
     StatsScopeId wfScopeId(SSTworkflow, wfid);
     StatsScopeId sgScopeId(SSTsubgraph, sgId);
-    statsCollection->recordStats(mapping, sourceStats, {wfScopeId, graphScopeId, sgScopeId});
+    statsCollection->recordStats(mapping, sourceStats, {std::move(wfScopeId), std::move(graphScopeId), std::move(sgScopeId)});
 }
 
 // Recalculate aggregates and then write the aggregates to global stats (dali)

diff --git a/system/jlib/jstats.cpp b/system/jlib/jstats.cpp
@@ -2242,7 +2242,7 @@ class CStatisticCollection : public CInterfaceOf<IStatisticCollection>
             ++scopeItem;
         }
 
-        CStatisticCollection * tgtScopeCollection = ensureSubScopePath(path);
+        CStatisticCollection * tgtScopeCollection = ensureSubScopePath(std::move(path));
         bool wasUpdated = false;
         // More efficient to iterate over stats rather than mapping...
         ForEachItemIn(i, curSrcCollection->stats)