[buddypress-trac] [BuddyPress Trac] #6220: Cache BP_Messages_Thread::get_recipients() to reduce inbox queries by 20

[buddypress-trac]

#6220: Cache BP_Messages_Thread::get_recipients() to reduce inbox queries by 20
-----------------------------------+------------------------------
 Reporter:  wpdennis               |       Owner:
     Type:  enhancement            |      Status:  new
 Priority:  normal                 |   Milestone:  Awaiting Review
Component:  Component - Messaging  |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  dev-feedback           |
-----------------------------------+------------------------------

Comment (by wpdennis):

 While digging deeper, I opened a second ticket: #6221

 I think we have two issues here:

 a) ''messages-loop.php'' calls things in a way, where redundant code (with
 db queries) is executed twice. This could be worth a look on its own and
 if so, #6221 would be the better ticket to adress this issue.

 b) ''get_recipients()'' should be trivial to cache, since the data is
 really persistant. So we could save at minimum 10 queries in any scenario
 and in the current implementation even 20 queries. So we could keep this
 ticket just for the caching of ''get_recipient()'', despite possible
 changes to ''messages-loop.php''.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6220#comment:1>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac