[#6658] Activity - Notification Icon slowing down user profile view

Hi,

When a user views there own profile, there is a noticeable delay in the page loading and this is due to the target user id being concatenated in the asset column.
My activity table is still quite small, under 10k rows and I expect it to grow, so this concerns me.

Please consider moving the target user id out of the asset column and create a new indexed column for it.

Thanks
David

I should add, I have been generating notifications, the current count is low:

The activity queries were tested against a table with 100,000 rows. They all performed perfectly fine taking less than 50ms. As the asset does not begin with a wildcard it can be indexed perfectly fine and does so in my tests. There is no concat of the user id. There is a concat done for the read check and during CB Gallery access check.

Will review the performance further, but you're basically using a feature that's not entirely done as I haven't decided if I want to make a new table for notifications or keep them in activity; it's creating some optimization problems as you've noticed.

forge.joomlapolis.com/issues/6658

Ok, put some thought into my previous reply. I'm going to split notifications into their own database table to help optimize it further. This would remove the need of "notification.USER_ID." prefixing the asset (a column will be added to designate the recipient user). Read table will also be removed entirely as we'll just track read state in the notification table it self. I'll add migration to migrate existing notification activity to the new table so nothing to worry about in that regard.

Sounds great, I will be watching out for the change as I will have some Auto Actions to update.

Thanks

Won't be for awhile as I'm working through other plugins before coming back to CB Activity and CB Auto Actions. You can continue to use it as is and should have migration in place when the change is made so all your existing usages can migrate.