Don't check execution first

dockstore · Jan 3, 2024 · 3d8f785 · 3d8f785
1 parent 41fdc27
commit 3d8f785
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/...egator/src/main/java/io/dockstore/metricsaggregator/client/cli/TerraMetricsSubmitter.java b/...egator/src/main/java/io/dockstore/metricsaggregator/client/cli/TerraMetricsSubmitter.java
@@ -156,12 +156,6 @@ private void processWorkflowExecution(CSVRecord workflowMetricRecord, WorkflowsA
             return;
         }
 
-        // Check if the information from the workflow execution is valid
-        Optional<RunExecution> workflowExecution = getTerraWorkflowExecutionFromCsvRecord(workflowMetricRecord, sourceUrl, skippedExecutionsCsvPrinter);
-        if (workflowExecution.isEmpty()) {
-            return;
-        }
-
         if (!sourceUrlToSourceUrlTrsInfo.containsKey(sourceUrl)) {
             Optional<SourceUrlTrsInfo> sourceUrlTrsInfo = calculateTrsInfoFromSourceUrl(workflowMetricRecord, sourceUrl, workflowsApi, skippedExecutionsCsvPrinter);
             if (sourceUrlTrsInfo.isEmpty()) {
@@ -171,6 +165,12 @@ private void processWorkflowExecution(CSVRecord workflowMetricRecord, WorkflowsA
             }
         }
 
+        // Check if the information from the workflow execution is valid
+        Optional<RunExecution> workflowExecution = getTerraWorkflowExecutionFromCsvRecord(workflowMetricRecord, sourceUrl, skippedExecutionsCsvPrinter);
+        if (workflowExecution.isEmpty()) {
+            return;
+        }
+
         final SourceUrlTrsInfo sourceUrlTrsInfo = sourceUrlToSourceUrlTrsInfo.get(sourceUrl);
         final ExecutionsRequestBody executionsRequestBody = new ExecutionsRequestBody().runExecutions(List.of(workflowExecution.get()));
         try {