Don't cancel TimerTask just because we had a network issue

komoot · Apr 10, 2019 · 8976c54 · 8976c54
1 parent a5f4ecc
commit 8976c54
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/main/java/de/komoot/photon/elasticsearch/Replicatior.java b/src/main/java/de/komoot/photon/elasticsearch/Replicatior.java
@@ -74,7 +74,7 @@ public void start() {
     /**
      * Read the local and remote state file and then download any missing replication files
      * 
-     * FIXME: should likely throw and exception if the difference between sequence numbers is very large
+     * FIXME: should likely throw an exception if the difference between sequence numbers is very large
      *
      */
     class ReplicationTask extends TimerTask {
@@ -132,8 +132,7 @@ public void run() {
                     log.info(String.format("Update done. %d deletions %d updates or additions.", deletions, updates));
                 }
             } catch (IOException e) {
-                log.error("Replicator failing", e);
-                cancel();
+                log.error("Replicator failing because ", e);
             }
         }
     }