From a023de3d28ccc9c37bdf7998fe02fb85f6596daf Mon Sep 17 00:00:00 2001 From: Athou Date: Sun, 30 Jun 2013 07:44:59 +0200 Subject: [PATCH] look subscriptions up only if needed --- .../com/commafeed/backend/feeds/FeedRefreshUpdater.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/main/java/com/commafeed/backend/feeds/FeedRefreshUpdater.java b/src/main/java/com/commafeed/backend/feeds/FeedRefreshUpdater.java index 1376576e..f7ce0b32 100644 --- a/src/main/java/com/commafeed/backend/feeds/FeedRefreshUpdater.java +++ b/src/main/java/com/commafeed/backend/feeds/FeedRefreshUpdater.java @@ -140,14 +140,19 @@ public class FeedRefreshUpdater { public void run() { boolean ok = true; if (entries.isEmpty() == false) { - List subscriptions = feedSubscriptionDAO - .findByFeed(feed); + List lastEntries = cache.getLastEntries(feed); List currentEntries = Lists.newArrayList(); + + List subscriptions = null; for (FeedEntry entry : entries) { String cacheKey = cache.buildKey(feed, entry); if (!lastEntries.contains(cacheKey)) { log.debug("cache miss for {}", entry.getUrl()); + if (subscriptions == null) { + subscriptions = feedSubscriptionDAO + .findByFeed(feed); + } ok &= updateEntry(feed, entry, subscriptions); metricsBean.entryCacheMiss(); } else {