garrettmills/commafeed
1
0
Fork 0
forked from Archives/Athou_commafeed
Code Pull Requests Activity

look subscriptions up only if needed

Browse Source
This commit is contained in:
Athou
2013-06-30 07:44:59 +02:00
parent aa55cb2a5d
commit a023de3d28
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/main/java/com/commafeed/backend/feeds/FeedRefreshUpdater.java
View File

@@ -140,14 +140,19 @@ public class FeedRefreshUpdater {
public void run() { public void run() {
boolean ok = true; boolean ok = true;
if (entries.isEmpty() == false) { if (entries.isEmpty() == false) {
List<FeedSubscription> subscriptions = feedSubscriptionDAO
.findByFeed(feed);
List<String> lastEntries = cache.getLastEntries(feed); List<String> lastEntries = cache.getLastEntries(feed);
List<String> currentEntries = Lists.newArrayList(); List<String> currentEntries = Lists.newArrayList();
List<FeedSubscription> subscriptions = null;
for (FeedEntry entry : entries) { for (FeedEntry entry : entries) {
String cacheKey = cache.buildKey(feed, entry); String cacheKey = cache.buildKey(feed, entry);
if (!lastEntries.contains(cacheKey)) { if (!lastEntries.contains(cacheKey)) {
log.debug("cache miss for {}", entry.getUrl()); log.debug("cache miss for {}", entry.getUrl());
if (subscriptions == null) {
subscriptions = feedSubscriptionDAO
.findByFeed(feed);
}
ok &= updateEntry(feed, entry, subscriptions); ok &= updateEntry(feed, entry, subscriptions);
metricsBean.entryCacheMiss(); metricsBean.entryCacheMiss();
} else { } else {