minor fix to log

dockstore · Feb 21, 2024 · 471f5ad · 471f5ad
1 parent cd4150f
commit 471f5ad
Showing 1 changed file with 3 additions and 1 deletion.
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
@@ -199,7 +199,6 @@ private void executionMetricsPost(List<RunExecution> workflowExecutionsToSubmit,
         try {
             extendedGa4GhApi.executionMetricsPost(new ExecutionsRequestBody().runExecutions(workflowExecutionsToSubmit), Partner.TERRA.toString(), sourceUrlTrsInfo.trsId(),
                     sourceUrlTrsInfo.version(), description);
-            numberOfExecutionsSubmitted.addAndGet(workflowMetricRecords.size());
         } catch (ApiException e) {
             if (e.getCode() == HttpStatus.SC_REQUEST_TOO_LONG) {
                 // One execution is too large, not much that can be done, so log and skip it
@@ -225,7 +224,10 @@ private void executionMetricsPost(List<RunExecution> workflowExecutionsToSubmit,
                         String.format("Could not submit execution metrics to Dockstore for workflow %s: %s", sourceUrlTrsInfo,
                                 e.getMessage()), skippedExecutionsCsvPrinter, false);
             }
+            return;
         }
+        // only count if there was no exception
+        numberOfExecutionsSubmitted.addAndGet(workflowMetricRecords.size());
     }
 
     /**