Merge pull request #411 from NASA-AMMOS/hotfix/sim-results-transactio…

…n-context Wrap simulation results writing in TransactionContext
NASA-AMMOS · Nov 2, 2022 · c14bcaf · c14bcaf
2 parents af8464c + b820e2f
commit c14bcaf
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/...java/gov/nasa/jpl/aerie/merlin/server/remotes/postgres/PostgresResultsCellRepository.java b/...java/gov/nasa/jpl/aerie/merlin/server/remotes/postgres/PostgresResultsCellRepository.java
@@ -614,8 +614,10 @@ public boolean isCanceled() {
 
     @Override
     public void succeedWith(final SimulationResults results) {
-      try (final var connection = dataSource.getConnection()) {
+      try (final var connection = dataSource.getConnection();
+           final var transactionContext = new TransactionContext(connection)) {
         postSimulationResults(connection, datasetId, results);
+        transactionContext.commit();
       } catch (final SQLException ex) {
         throw new DatabaseException("Failed to store simulation results", ex);
       } catch (final NoSuchSimulationDatasetException ex) {