Skip to content

Commit

Permalink
Don't cancel TimerTask just because we had a network issue
Browse files Browse the repository at this point in the history
  • Loading branch information
simonpoole committed Mar 1, 2019
1 parent 7f0e701 commit d5b1953
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions src/main/java/de/komoot/photon/elasticsearch/Replicatior.java
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down Expand Up @@ -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);
}
}
}
Expand Down

0 comments on commit d5b1953

Please sign in to comment.