Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Course enrollment admin is slow #736

Open
MaferMazu opened this issue Feb 28, 2023 · 7 comments
Open

Course enrollment admin is slow #736

MaferMazu opened this issue Feb 28, 2023 · 7 comments
Assignees

Comments

@MaferMazu
Copy link
Contributor

MaferMazu commented Feb 28, 2023

Description

"Course enrollment admin is slow" was a report from SD after migration.

Talking with @Ian2012, he thinks the problem is in the ordering. By default, the query order with user_id is 97secs, and without order, it takes 70secs.

We need to find where that class is ordered and modify it to solve this. If the product review accepts this, I don't know if we can extract this from the custom features in edx-platform to some eox or if we need to maintain this.

Extra info: https://user-images.githubusercontent.com/35668326/221997715-8ffabc76-bef7-46fb-8d93-4ce1cfd5dbc8.png

@santiagosuarezedunext
Copy link

@Ian2012 me podrias explicar un poco más sobre que es "Course enrollment admin" y el problema por favor 👍

@Ian2012
Copy link
Member

Ian2012 commented Mar 1, 2023

Cuando se carga la tabla course enrollment en el panel de administración, este carga demasiado lento. Todo indica que es por el ordering que se aplica a esa tabla, en lilac los registros no se ordenaban, ahora si se ordenan y carga muy lento

@santiagosuarezedunext
Copy link

@MaferMazu ¿esto hace parte de las cosas custom que mantenemos o entendí mal? Si si hace parte ¿deberiamos registrarla en nuestra tabla de deprecaciones y darle proceso por ahí?

@MaferMazu
Copy link
Contributor Author

Es un feature request. Es algo que está solicitando service delivery, es algo nuevo.

@santiagosuarezedunext
Copy link

@MaferMazu what do you mean with this: " I don't know if we can extract this from the custom features in edx-platform to some eox or if we need to maintain this."

@MaferMazu
Copy link
Contributor Author

It means I don't know if we can implement a solution for this somewhere different from edunext-platform. If we can't that means adding a new custom feature to maintain in the list we try to empty.
Maybe a good idea is to avoid adding this to maintain and try to push the solution upstream. We can try to find out why the edx-platform is ordering, and propose not to order (but we need strong arguments).

@santiagosuarezedunext
Copy link

Okay, I think if we make the change we should make it upstream.

@Ian2012
Since this doesn't drastically affect anyone, upstreaming it can be delayed and right now we are working on upstreaming other features, i'm going to move this to the backlog and we'll take it when we have time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants