From 39814d1efd9bb7b4b81a3a87c003a2aa203ccd0e Mon Sep 17 00:00:00 2001
From: Noah Levitt <nlevitt@archive.org>
Date: Thu, 22 May 2014 16:40:44 -0700
Subject: [PATCH] avoid calling CheckpointService.hasAvailableCheckpoints()
 when crawl not launchable and therefore the info is not needed, because it
 calls isRunning() which is synchronized and blocks when checkpoint is in
 progress

---
 .../java/org/archive/crawler/restlet/models/CrawlJobModel.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/engine/src/main/java/org/archive/crawler/restlet/models/CrawlJobModel.java b/engine/src/main/java/org/archive/crawler/restlet/models/CrawlJobModel.java
index 0e02be0e..1ea8d871 100644
--- a/engine/src/main/java/org/archive/crawler/restlet/models/CrawlJobModel.java
+++ b/engine/src/main/java/org/archive/crawler/restlet/models/CrawlJobModel.java
@@ -127,7 +127,7 @@ public class CrawlJobModel extends LinkedHashMap<String, Object> implements Seri
         
         List<String> checkpointFiles = new ArrayList<String>();
         if (crawlJob.getCheckpointService() != null) {
-            if (crawlJob.getCheckpointService().hasAvailableCheckpoints() && crawlJob.isLaunchable()) {
+            if (crawlJob.isLaunchable() && crawlJob.getCheckpointService().hasAvailableCheckpoints()) {
                 for (File f : crawlJob.getCheckpointService().findAvailableCheckpointDirectories()) {
                     checkpointFiles.add(f.getName()); 
                 }