Move warning inside check condition (#242)

Author: chuck-dbos

transact/src/main/java/dev/dbos/transact/database/StepsDAO.java

@@ -94,11 +94,11 @@ static void recordStepResultTxn(
       try (ResultSet rs = pstmt.executeQuery()) {
         if (rs.next() && endTimeEpochMs != null) {
           long completedAt = rs.getLong("completed_at_epoch_ms");
-          logger.warn(
-              String.format(
-                  "Step output for %s:%d-%s was already recorded",
-                  result.workflowId(), result.stepId(), result.functionName()));
           if (completedAt != endTimeEpochMs) {
+            logger.warn(
+                String.format(
+                    "Step output for %s:%d-%s was already recorded",
+                    result.workflowId(), result.stepId(), result.functionName()));
             throw new DBOSWorkflowExecutionConflictException(result.workflowId());
           }
         }