[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:  Awaiting Review
Component:  Core          |    Version:
 Severity:  normal        |   Keywords:
--------------------------+-----------------------------
 Hi there, we have more than 1.6M users on our database. Around 700.000
 records on wp_bp_notifications. We use
 https://tr.wordpress.org/plugins/memcached/ for object caching.

 Some users have 37k wp_bp_notifications record, another 20k, another 17k,
 11k and goes on.

 When we enter bp profile page of these users
 bp_notifications_get_all_notifications_for_user runs and tries to get all
 these records, and it can get but wp_cache_set can not set cache. It gives
 that error:

 PHP Fatal error:  Allowed memory size of 134217728 bytes exhausted (tried
 to allocate 2097152 bytes) in /var/www/wp-content/object-cache.php on line
 308

 If user have a lot of notification record it gives that error, if users
 doesnt have much record there is no problem.

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