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

Enhance feedback response result endpoint #11011

Open
moziliar opened this issue Mar 5, 2021 · 0 comments
Open

Enhance feedback response result endpoint #11011

moziliar opened this issue Mar 5, 2021 · 0 comments
Assignees
Labels
a-Performance Speed and efficiency of operations a-Scalability Behaviour at increasing/decreasing loads committers only Difficult; better left for committers or more senior developers

Comments

@moziliar
Copy link
Contributor

moziliar commented Mar 5, 2021

This ticket is created to track the status of the following issue:

Instructor results page requires all responses from all students in a course in a feedback session to be displayed. As the course/feedback session size grows, this becomes a problem when loading the responses as it can cause either out-of-memory (OOM) error (all responses need to be loaded in the back-end before transferred to front-end) or timeout (due to the large amount of entities that need to be read).

There are several issues to tackle in this issue:

1. Backend fine-grained response fetch to prevent OOM or server timeout [TODO]
2. Frontend response result loading optimization, related to above [TODO]
3. Frontend CSV download improvement via pagination [IN PROGRESS]
4. Workflow cost analysis [TODO]

Issue 1 and 2 should have a finer grained fetching logic of the student response. We could employ lazy loading (given that the instructor cannot view all the results at once in large course setting) with more smaller api calls. This is open for discussion.

Issue 3 could capitalize on solution to 1 and 2 while staying paginated to smoothen backend memory pressure.

Issue 4 is the overall cost calculation for the improvement.

@moziliar moziliar added a-Performance Speed and efficiency of operations committers only Difficult; better left for committers or more senior developers p.High Significant impact; would like to do in the next few releases t-Angular a-Scalability Behaviour at increasing/decreasing loads labels Mar 5, 2021
@moziliar moziliar removed the p.High Significant impact; would like to do in the next few releases label Mar 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-Performance Speed and efficiency of operations a-Scalability Behaviour at increasing/decreasing loads committers only Difficult; better left for committers or more senior developers
Projects
None yet
Development

No branches or pull requests

4 participants