[buddypress-trac] [BuddyPress Trac] #7130: bp_notifications_get_all_notifications_for_user gets all records and can cause memory leak

[buddypress-trac]

#7130: bp_notifications_get_all_notifications_for_user gets all records and can
cause memory leak
-------------------------------------+------------------
 Reporter:  hberberoglu              |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  3.0
Component:  Toolbar & Notifications  |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch                |
-------------------------------------+------------------

Comment (by m_uysl):

 > Can you look at the new patch to see if I've captured it correctly?
 @boonebgorges looks good to me.

 I've tested patches with ~24k notifications and ~98M memory usage reduced
 to ~36M, page generation time improved as well.

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