From 4ff26366a54630ac5bd97d9e54089ba720aa9e85 Mon Sep 17 00:00:00 2001 From: Athou Date: Mon, 1 May 2023 10:04:43 +0200 Subject: [PATCH] there's no need to update disabledUntil here anymore because findNextUpdatableFeeds will always be called when the queue is empty --- .../commafeed/backend/service/FeedRefreshEngine.java | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/commafeed-server/src/main/java/com/commafeed/backend/service/FeedRefreshEngine.java b/commafeed-server/src/main/java/com/commafeed/backend/service/FeedRefreshEngine.java index c570fcfc..909a0d25 100644 --- a/commafeed-server/src/main/java/com/commafeed/backend/service/FeedRefreshEngine.java +++ b/commafeed-server/src/main/java/com/commafeed/backend/service/FeedRefreshEngine.java @@ -92,17 +92,7 @@ public class FeedRefreshEngine implements Managed { private List findNextUpdatableFeeds(int max, Date lastLoginThreshold) { refill.mark(); - - return UnitOfWork.call(sessionFactory, () -> { - List list = feedDAO.findNextUpdatable(max, lastLoginThreshold); - - // set the disabledDate as we use it in feedDAO.findNextUpdatable() to decide what to refresh next - Date nextRefreshDate = DateUtils.addMinutes(new Date(), config.getApplicationSettings().getRefreshIntervalMinutes()); - list.forEach(f -> f.setDisabledUntil(nextRefreshDate)); - feedDAO.saveOrUpdate(list); - - return list; - }); + return UnitOfWork.call(sessionFactory, () -> feedDAO.findNextUpdatable(max, lastLoginThreshold)); } private int getBatchSize() {