perf(Trace): optimise getTotalTimeDurationInMillis function to push t…

…o an array instead of concat arrays. (#1774) * Thanks @infinity-naveen for your contribution 🎉
newrelic · Sep 1, 2023 · 6ecde97 · 6ecde97
1 parent 3f494f8
commit 6ecde97
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/transaction/trace/index.js b/lib/transaction/trace/index.js
@@ -218,13 +218,13 @@ Trace.prototype.getTotalTimeDurationInMillis = function getTotalTimeDurationInMi
   if (this.root.children.length === 0) {
     return 0
   }
-  let segments = this.root.getChildren()
+  const segments = this.root.getChildren()
   let totalTimeInMillis = 0
 
   while (segments.length !== 0) {
     const segment = segments.pop()
     totalTimeInMillis += segment.getExclusiveDurationInMillis()
-    segments = segments.concat(segment.getChildren())
+    segment.getChildren().forEach((childSegment) => segments.push(childSegment))
   }
 
   if (!this.transaction.isActive()) {