Remove endTime on retried runs

ERROR runs should have an endTime, RUNNING runs shouldn't.
thelastpickle · May 13, 2015 · 30f6a31 · 30f6a31
1 parent 228381a
commit 30f6a31
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/main/java/com/spotify/reaper/service/RepairManager.java b/src/main/java/com/spotify/reaper/service/RepairManager.java
@@ -109,9 +109,10 @@ public RepairRun startRepairRun(AppContext context, RepairRun runToBeStarted) {
         LOG.info("re-trigger a running run after restart, with id " + runId);
         startRunner(context, runId);
         return runToBeStarted;
-      case ERROR:  {
+      case ERROR: {
         RepairRun updatedRun = runToBeStarted.with()
             .runState(RepairRun.RunState.RUNNING)
+            .endTime(null)
             .build(runToBeStarted.getId());
         if (!context.storage.updateRepairRun(updatedRun)) {
           throw new RuntimeException("failed updating repair run " + updatedRun.getId());