removed as it in fact hurts perfs

This commit is contained in:
Athou
2013-06-05 18:37:24 +02:00
parent 6eaa6d2712
commit 39f1a19c14

View File

@@ -33,7 +33,6 @@ import com.commafeed.backend.model.FeedSubscription;
import com.commafeed.backend.pubsubhubbub.SubscriptionHandler;
import com.commafeed.backend.services.ApplicationSettingsService;
import com.commafeed.backend.services.FeedUpdateService;
import com.google.common.collect.Lists;
import com.google.common.util.concurrent.Striped;
@Singleton
@@ -121,7 +120,6 @@ public class FeedRefreshUpdater {
@Override
public void run() {
boolean ok = true;
filterOldEntries();
if (entries.isEmpty() == false) {
List<FeedSubscription> subscriptions = feedSubscriptionDAO
.findByFeed(feed);
@@ -139,31 +137,6 @@ public class FeedRefreshUpdater {
metricsBean.feedUpdated();
taskGiver.giveBack(feed);
}
private void filterOldEntries() {
if (entries != null && entries.size() > 0 && entries.size() <= 50) {
List<FeedEntry> keep = Lists.newArrayList();
List<String> guids = Lists.newArrayList();
for (FeedEntry entry : entries) {
guids.add(entry.getGuid());
}
List<FeedEntry> existingEntries = feedEntryDAO
.findByGuids(guids);
for (FeedEntry entry : entries) {
FeedEntry found = FeedUtils.findEntry(existingEntries,
entry);
if (found == null
|| FeedUtils.findFeed(found.getFeeds(), feed) == null) {
keep.add(entry);
}
}
entries = keep;
}
}
}
private boolean updateEntry(final Feed feed, final FeedEntry entry,