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

Student index page / TestStudentIndexView performance #2235

Open
richardebeling opened this issue Jun 30, 2024 · 0 comments
Open

Student index page / TestStudentIndexView performance #2235

richardebeling opened this issue Jun 30, 2024 · 0 comments
Labels
[C] Backend Focuses on backend implementation [P] Medium Medium priority [T] Refactoring Existing parts should become faster, more readable, or in any other way better.

Comments

@richardebeling
Copy link
Member

TestStudentIndexView.test_num_queries_is_constant already is pretty slow (2s on my machine) and becomes even slower when increasing the constant 100 evaluations to 1000.

Interrupting the computation, the traceback indicated that the code was stuck in retrieving the course_id_evaluation_weight_sum_pairs in get_evaluations_with_course_result_attributes. This should be a pretty simple and fast query. Maybe the Course subquery isn't optimized out properly by postgres.

In general, the student index page shouldn't load longer than the results index page, but it currently does (1.4s for me vs 0.6s on results)

@janno42 janno42 added [C] Backend Focuses on backend implementation [P] Medium Medium priority [T] Refactoring Existing parts should become faster, more readable, or in any other way better. labels Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] Backend Focuses on backend implementation [P] Medium Medium priority [T] Refactoring Existing parts should become faster, more readable, or in any other way better.
Development

No branches or pull requests

2 participants