Lift timing out of hot loop

apache · Sep 8, 2022 · 8143bb9 · 8143bb9
1 parent 0ddcc50
commit 8143bb9
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/datafusion/core/src/physical_plan/sorts/sort_preserving_merge.rs b/datafusion/core/src/physical_plan/sorts/sort_preserving_merge.rs
@@ -545,12 +545,12 @@ impl SortPreservingMergeStream {
             }
         }
 
-        loop {
-            // NB timer records time taken on drop, so there are no
-            // calls to `timer.done()` below.
-            let elapsed_compute = self.tracking_metrics.elapsed_compute().clone();
-            let _timer = elapsed_compute.timer();
+        // NB timer records time taken on drop, so there are no
+        // calls to `timer.done()` below.
+        let elapsed_compute = self.tracking_metrics.elapsed_compute().clone();
+        let _timer = elapsed_compute.timer();
 
+        loop {
             match self.heap.pop() {
                 Some(Reverse(mut cursor)) => {
                     let stream_idx = cursor.stream_idx();