From cadb64858a36ab93d1be48636629bc7e42ad9d66 Mon Sep 17 00:00:00 2001 From: Kathy Tran Date: Mon, 26 Feb 2024 10:03:01 -0500 Subject: [PATCH] Add clarification --- .../metricsaggregator/helper/RunExecutionAggregator.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/metricsaggregator/src/main/java/io/dockstore/metricsaggregator/helper/RunExecutionAggregator.java b/metricsaggregator/src/main/java/io/dockstore/metricsaggregator/helper/RunExecutionAggregator.java index abaafd98..a96a1f0d 100644 --- a/metricsaggregator/src/main/java/io/dockstore/metricsaggregator/helper/RunExecutionAggregator.java +++ b/metricsaggregator/src/main/java/io/dockstore/metricsaggregator/helper/RunExecutionAggregator.java @@ -23,7 +23,8 @@ public List getTaskExecutionsWithMetric(List tas } /** - * Aggregates TaskExecutions that belong to a single workflow run into a workflow-level RunExecution. Does NOT check that the resulting workflow run is valid. + * Aggregates TaskExecutions that belong to a single workflow run into a workflow-level RunExecution. + * Does NOT check that the resulting workflow run is valid. The validity check is done when workflow executions are aggregated so that the aggregator can recognize that task metrics were skipped. * @param taskExecutionsForOneWorkflowRun * @return */