ComputerScienceHouse / ComputerScienceHouse/packet

/admin/freshmen Times Out

未关闭
#261 1 条评论 0 个 reaction 已指派 1 人 已被 @VivianNK 认领 在 GitHub 查看
bug good first issue performance
主要语言
Python
星标
25
派生
33
PR 合并指标
30 天内没有已合并 PR

描述

The `/admin/freshmen` route times out when there are lots of freshmen (~300 in prod, much more in dev). It works running locally with only two freshmen, so the issue seems to be tied to the amount of data that's being processed and rendered.

My guess is that pagination or lazy loading would solve this.

For pagination, I recommend the [flask sqlalchemy paginate function](https://flask-sqlalchemy.palletsprojects.com/en/2.x/api/?highlight=paginate#flask_sqlalchemy.BaseQuery.paginate).

Lazy loading is a little more complex, and might need api changes. TL;DR: rather than loading up all the freshmen, then rendering the page, the lazy load version would defer the data loading till after the page is rendered, so the user could interact with the page while it's still being populated. (I'm probably not using the term quite correctly here, but it's close enough to the idea)

贡献指南

这个仓库没有索引到贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。