Skip to content

Commit 67f2644

Browse files
committed
fixup: use bench_by_ts
1 parent 8e94b32 commit 67f2644

2 files changed

Lines changed: 3 additions & 19 deletions

File tree

crates/runner-shared/src/metadata.rs

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,6 @@
1-
// !!!!!!!!!!!!!!!!!!!!!!!!
2-
// !! DO NOT TOUCH BELOW !!
3-
// !!!!!!!!!!!!!!!!!!!!!!!!
4-
// Has to be in sync with `perf-parser`.
5-
//
6-
71
use anyhow::Context;
82
use serde::{Deserialize, Serialize};
9-
use std::{collections::HashMap, path::Path};
3+
use std::path::Path;
104

115
use crate::fifo::MarkerType;
126

@@ -21,7 +15,7 @@ pub struct PerfMetadata {
2115
pub integration: (String, String),
2216

2317
/// The URIs of the benchmarks in the order they were executed.
24-
pub bench_order_by_pid: HashMap<u32, Vec<String>>,
18+
pub bench_by_ts: Vec<(u64, String)>,
2519

2620
/// Modules that should be ignored and removed from the folded trace and callgraph (e.g. python interpreter)
2721
pub ignored_modules: Vec<(String, u64, u64)>,

src/run/runner/wall_time/perf/mod.rs

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -392,17 +392,7 @@ impl BenchmarkData {
392392
.integration
393393
.clone()
394394
.ok_or(BenchmarkDataSaveError::MissingIntegration)?,
395-
bench_order_by_ts: {
396-
let mut bench_order_by_ts = HashMap::<u32, Vec<String>>::new();
397-
// Group all benchmarks under PID 0 since we don't track PID anymore
398-
let uris: Vec<String> = self
399-
.bench_order_by_timestamp
400-
.iter()
401-
.map(|(_, uri)| uri.clone())
402-
.collect();
403-
bench_order_by_ts.insert(0, uris);
404-
bench_order_by_ts
405-
},
395+
bench_by_ts: self.bench_order_by_timestamp.clone(),
406396
ignored_modules: {
407397
let mut to_ignore = vec![];
408398

0 commit comments

Comments
 (0)