Archives/Athou_commafeed
1
0
Fork 1
mirror of https://github.com/Athou/commafeed.git synced 2026-03-21 21:37:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

query optimizations

Browse Source
This commit is contained in:
Jeremie Panzer
2013-03-27 10:46:30 +01:00
parent 4882c39aca
commit f5a32fce13
4 changed files with 8 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/main/java/com/commafeed/backend/dao/FeedSubscriptionService.java
View File

@@ -6,7 +6,6 @@ import javax.ejb.Stateless;
import javax.inject.Inject; import javax.inject.Inject;
import com.commafeed.backend.model.Feed; import com.commafeed.backend.model.Feed;
import com.commafeed.backend.model.FeedCategory;
import com.commafeed.backend.model.FeedSubscription; import com.commafeed.backend.model.FeedSubscription;
import com.commafeed.backend.model.User; import com.commafeed.backend.model.User;
import com.commafeed.frontend.utils.ModelFactory.MF; import com.commafeed.frontend.utils.ModelFactory.MF;
@@ -36,14 +35,14 @@ public class FeedSubscriptionService extends GenericDAO<FeedSubscription, Long>
} }
public List<FeedSubscription> findAll(User user) { public List<FeedSubscription> findAll(User user) {
return findByField(MF.i(MF.p(FeedCategory.class).getUser()), user); return findByField(MF.i(proxy().getUser()), user);
} }
public List<FeedSubscription> findWithoutCategories(User user) { public List<FeedSubscription> findWithoutCategories(User user) {
EasyCriteria<FeedSubscription> criteria = EasyCriteriaFactory EasyCriteria<FeedSubscription> criteria = EasyCriteriaFactory
.createQueryCriteria(em, getType()); .createQueryCriteria(em, getType());
criteria.andEquals("user", user); criteria.andEquals(MF.i(proxy().getUser()), user);
criteria.andIsNull("category"); criteria.andIsNull(MF.i(proxy().getCategory()));
return criteria.getResultList(); return criteria.getResultList();
} }

3
src/main/java/com/commafeed/backend/model/FeedEntry.java
View File

@@ -5,7 +5,6 @@ import java.util.Set;
import javax.persistence.Column; import javax.persistence.Column;
import javax.persistence.Entity; import javax.persistence.Entity;
import javax.persistence.FetchType;
import javax.persistence.Lob; import javax.persistence.Lob;
import javax.persistence.ManyToOne; import javax.persistence.ManyToOne;
import javax.persistence.OneToMany; import javax.persistence.OneToMany;
@@ -23,7 +22,7 @@ public class FeedEntry extends AbstractModel {
@Column(length = 2048) @Column(length = 2048)
private String guid; private String guid;
@ManyToOne(fetch = FetchType.EAGER) @ManyToOne
private Feed feed; private Feed feed;
@Column(length = 2048) @Column(length = 2048)

5
src/main/java/com/commafeed/backend/model/FeedSubscription.java
View File

@@ -2,7 +2,6 @@ package com.commafeed.backend.model;
import javax.persistence.Column; import javax.persistence.Column;
import javax.persistence.Entity; import javax.persistence.Entity;
import javax.persistence.FetchType;
import javax.persistence.ManyToOne; import javax.persistence.ManyToOne;
import javax.persistence.Table; import javax.persistence.Table;
@@ -11,10 +10,10 @@ import javax.persistence.Table;
@SuppressWarnings("serial") @SuppressWarnings("serial")
public class FeedSubscription extends AbstractModel { public class FeedSubscription extends AbstractModel {
@ManyToOne(fetch = FetchType.EAGER) @ManyToOne
private User user; private User user;
@ManyToOne(fetch = FetchType.EAGER) @ManyToOne
private Feed feed; private Feed feed;
@Column(length = 128) @Column(length = 128)

5
src/main/java/com/commafeed/frontend/rest/resources/EntriesREST.java
View File

@@ -61,8 +61,7 @@ public class EntriesREST extends AbstractREST {
} else { } else {
FeedCategory feedCategory = null; FeedCategory feedCategory = null;
if (!ALL.equals(id)) { if (!ALL.equals(id)) {
feedCategory = new FeedCategory(); feedCategory = feedCategoryService.findById(getUser(), Long.valueOf(id));
feedCategory.setId(Long.valueOf(id));
} }
List<FeedCategory> childrenCategories = feedCategoryService List<FeedCategory> childrenCategories = feedCategoryService
.findAllChildrenCategories(getUser(), feedCategory); .findAllChildrenCategories(getUser(), feedCategory);
@@ -75,7 +74,7 @@ public class EntriesREST extends AbstractREST {
} }
}); });
entries.setName(ALL.equals(id) ? ALL : feedCategory.getName()); entries.setName(ALL.equals(id) ? "All" : feedCategory.getName());
entries.getEntries().addAll( entries.getEntries().addAll(
buildEntries(childrenCategories, subMapping, offset, limit, buildEntries(childrenCategories, subMapping, offset, limit,
unreadOnly)); unreadOnly));