Merge pull request #52 from nlnwa/maeb-patch-1

Fix offset calculation in NextUriScript.java
nlnwa · Nov 17, 2021 · bb99a51 · bb99a51
2 parents 866c7ca + 6f7423e
commit bb99a51
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/no/nb/nna/veidemann/frontier/db/script/NextUriScript.java b/src/main/java/no/nb/nna/veidemann/frontier/db/script/NextUriScript.java
@@ -60,8 +60,8 @@ public NextUriScriptResult run(JedisContext ctx, CrawlHostGroup crawlHostGroup)
             String key = UCHG + chgId;
             String minScore = String.valueOf(Math.random() * maxScore);
             Long matchCount = jedis.zcount(key, minScore, "+inf");
-            long offset = (int) (Math.random() * (matchCount - 1));
-            Set<String> eResult = jedis.zrangeByScore(key, minScore, "+inf", (int) offset, 1);
+            int offset = (int) (Math.random() * matchCount);
+            Set<String> eResult = jedis.zrangeByScore(key, minScore, "+inf", offset, 1);
             if (eResult.isEmpty()) {
                 return new NextUriScriptResult(FutureOptional.empty());
             }