[buddypress-trac] [BuddyPress] #4041: Use SELECT FOUND_ROWS() for 'total' queries instead of a second query

[buddypress-trac at lists.automattic.com]

#4041: Use SELECT FOUND_ROWS() for 'total' queries instead of a second query
----------------------------+-----------------------
 Reporter:  boonebgorges    |       Owner:  DJPaul
     Type:  defect (bug)    |      Status:  assigned
 Priority:  normal          |   Milestone:  1.6
Component:  All Components  |     Version:
 Severity:  normal          |  Resolution:
 Keywords:                  |
----------------------------+-----------------------

Comment (by boonebgorges):

 webraket - Thanks for the pointer. WordPress uses SELECT FOUND_ROWS() in
 WP_Query - have you found similar performance issues?

 I would like to benchmark this before moving forward with any broad
 changes. Someone should write a script that will write a few million
 records into the activities table (I will do it this weekend if I get a
 chance) and then test webraket's patch for the activity query against
 SELECT FOUND_ROWS(). If they are close or the same, we should go with
 SELECT FOUND_ROWS() (all things being equal, it's better to do what WP
 does, better to hand work off to the DB, better to have fewer queries).
 But if we find that webraket's results are reproducible, then we should
 probably think about keeping what we currently have in place, at least for
 components (like Activity) where the table is liable to get very, very,
 very large.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4041#comment:4>
BuddyPress <http://buddypress.org/>
BuddyPress