search

Automattic / sensei

Sensei LMS - Online Courses, Quizzes, & Learning
https://senseilms.com
GNU General Public License v2.0
539 stars 198 forks source link

Poor performance listing users when site has large number of users [L] #2215

Open dartiss opened 6 years ago

dartiss commented 6 years ago

A WordPress.com VIP client, with 45K users is using Sensei. When attempting to use the Courses menu (wp-admin/edit.php?post_type=course), it takes up to 2 minutes to display.

We have tracked the slow query down to https://github.com/Automattic/sensei/blob/b67cba4fc4f4bb73c2fb83309add46abd32c387a/includes/class-sensei-analysis-overview-list-table.php#L549.

Can somebody take a look at this to see if any performance improvements can be made when there are large numbers of users?

Thanks.

VIP ticket: #81444-z

sophosscott commented 6 years ago

Client here. For context: we're in the process of adding 30+ new courses in multiple languages to Sensei, meaning lots of page loads on the courses list, lesson list, course edit, and lesson edit pages. The performance issues, particularly on the courses list page, makes the current effort pretty painful.

sophosscott commented 6 years ago

Can anyone comment on this? It has become very time-consuming to manage content in Sensei at this point.

LukeAbell commented 6 years ago

Having the same issue. We're going to need to fork the plugin to fix the huge SQL queries that Sensei is doing to pull all products and users when loading lessons & courses. This needs to be ajax driven. We're using sensei on a few sites with 100k users + 20k products.

lkraav commented 6 years ago

Would a crowdfunded pull request effort make sense here? Core team is unlikely to be able to move with any haste on this, since there simply isn't enough resource.