From 5945dff8ed76970d7f994fcdaafe507c4a482051 Mon Sep 17 00:00:00 2001 From: Mihai Budiu Date: Wed, 1 Jul 2026 15:04:09 -0700 Subject: [PATCH] [profiler] merges.avg_step_time may be undefined Signed-off-by: Mihai Budiu --- js-packages/profiler-lib/src/profile.ts | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/js-packages/profiler-lib/src/profile.ts b/js-packages/profiler-lib/src/profile.ts index 506d9ff7bf1..f6bdd42ea68 100644 --- a/js-packages/profiler-lib/src/profile.ts +++ b/js-packages/profiler-lib/src/profile.ts @@ -998,16 +998,21 @@ export class Measurement { } case "merges": { let s = metric.value as MergesMetricValue; - let avg_step_time = TimeValue.fromDurationMetric(s.avg_step_time); + let avg_step_time = undefined; + let result = []; + if (s.avg_step_time !== undefined) { + avg_step_time = TimeValue.fromDurationMetric(s.avg_step_time); + result.push(new Measurement(metric_id + ".avg_step_time", Option.some(avg_step_time))); + } let batches = CountValue.fromCountMetric(s.batches); let merges = CountValue.fromCountMetric(s.merges); let steps = CountValue.fromCountMetric(s.steps); - return [ - new Measurement(metric_id + ".avg_step_time", Option.some(avg_step_time)), + result.push( new Measurement(metric_id + ".batches", Option.some(batches)), new Measurement(metric_id + ".merges", Option.some(merges)), new Measurement(metric_id + ".steps", Option.some(steps)), - ]; + ); + return result; } case "policy": { let s = metric.value as StringMetricValue;