From 288f6023a3c546e6020aaa09ac2e79c1e93081bf Mon Sep 17 00:00:00 2001
From: Iuliia Loikova <iuliia.loikova@foodora.com>
Date: Thu, 9 Nov 2017 23:31:30 +0100
Subject: [PATCH] Unassigned job reason tracker returns the last failed
 constraint instead of the most likely

---
 .../jsprit/core/util/UnassignedJobReasonTracker.java           | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/util/UnassignedJobReasonTracker.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/util/UnassignedJobReasonTracker.java
index 63dbf938..a4577a05 100644
--- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/util/UnassignedJobReasonTracker.java
+++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/util/UnassignedJobReasonTracker.java
@@ -32,12 +32,13 @@ public class UnassignedJobReasonTracker implements JobUnassignedListener {
     public static String getMostLikelyFailedConstraintName(Frequency failedConstraintNamesFrequency) {
         if (failedConstraintNamesFrequency == null) return "no reason found";
         Iterator<Map.Entry<Comparable<?>, Long>> entryIterator = failedConstraintNamesFrequency.entrySetIterator();
-        int maxCount = 0;
+        long maxCount = 0;
         String mostLikely = null;
         while (entryIterator.hasNext()) {
             Map.Entry<Comparable<?>, Long> entry = entryIterator.next();
             if (entry.getValue() > maxCount) {
                 Comparable<?> key = entry.getKey();
+                maxCount = entry.getValue();
                 mostLikely = key.toString();
             }
         }