Looking at blocks like the "Featured on this site" block on the front page and the main content view on the group homepage. If we need prev/next style pagers, it'd be nice if we could use views_litepager and avoid the COUNT queries.

IMO it's also a UX improvement as that space looks pretty cluttered as it is.

Comments

msonnabaum’s picture

To add a bit more data, I initially saw an issue on the /user page, in the "Following" block. It performs this query, which won't scale well as the user table grows:

SELECT COUNT(*) AS expression FROM (
SELECT 1 AS expression
FROM users users
INNER JOIN flag_content flag_content_users ON users.uid = flag_content_users.content_id AND flag_content_users.fid = '8'
WHERE (( (flag_content_users.uid = '9' ) )AND(( (users.status <> '0') )))
) subquery

Let's just avoid the subquery altogether and not have a typical pager there.

ezra-g’s picture

Issue tags: +Performance

Thanks! Adding the 'performance' tag.

ezra-g’s picture

I switched the Commons Follow User* views to be limited to 16 and user the "view more" pager, eliminating the count query.

http://drupalcode.org/project/commons_follow.git/commit/517ddc5

ezra-g’s picture

Title: Use litepager for views » Use litepager for BW views

Re-titling so we can be sure to target the views that are most likely to be subject to paging.

ezra-g’s picture

I added Views Litepager to the make file as first step here:

http://drupalcode.org/project/commons.git/commit/82a1d47

ezra-g’s picture

Priority: Normal » Major

Bumping priority to "major" so we can address this before 3.0.

Automatically closed -- issue fixed for 2 weeks with no activity.