Push filtering, sorting and pagination into the database layer
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
まず LearnStack.Core/Services/LearningResourceService.cs:12-21 と :74-96 を読み、次に Resources.razor:636-643 と Pulse.razor.cs:60 を調べます。query/filter と paged-result の設計を導入する前に、現在のインメモリ操作がどのように動作するかを追跡します。完了の条件は、リソース一覧の各ページにつきクエリが 1 回、重複 URL のインデックス付き検索が 1 回であり、Pulse の集計のためにライブラリ全体をマテリアライズしないことです。
索引モデルが issue の本文から書いたものです。
説明
Tier 2 — scale
Problem
GetAllAsync(userId) loads a user's entire library into memory (thumbnails included); the component then filters, sorts and pages it client-side. UrlExistsAsync loads every URL a user owns just to compare one. Pulse loads everything to compute aggregates. This is fine at 50 resources and collapses at 5,000.
Evidence
LearnStack.Core/Services/LearningResourceService.cs:12-21—GetAllAsyncLearnStack.Core/Services/LearningResourceService.cs:74-96—UrlExistsAsyncmaterializes all URLsLearnStack/Components/Pages/Resources.razor:636-643— in-memory pagingLearnStack/Components/Pages/Pulse.razor.cs:60— loads all resources for aggregates
Proposed fix
- Introduce a query/filter object and return a paged result (
Items,TotalCount) from the service. - Apply search, status/type/priority filters, sorting and
Skip/TakeinIQueryable. - Add a persisted
NormalizedUrlcolumn with a unique index per user, and replaceUrlExistsAsyncwith a single indexed lookup. - Compute Pulse aggregates with
GroupByin SQL rather than in memory.
Acceptance criteria
- Resource list issues one query per page regardless of library size
- Duplicate-URL check is a single indexed query
- Pulse does not materialize the full library
- 主要言語
- HTML
- スター
- 10
- フォーク
- 0
- 平均マージ
- 2時間 35分
- マージ済み PR（30日）
- 21
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
kasuken/LearnStack のほかの issue
-
enhancement
難易度 1/5 1時間未満 初心者へのやさしさ 91/100
kasuken/LearnStack#59 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
kasuken/LearnStack#72 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
kasuken/LearnStack#71 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
kasuken/LearnStack#70 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 45/100
kasuken/LearnStack#69 ·
メンテナーはふだん 1 日以内に返信
kasuken/LearnStack の issue をすべて見る
似ている issue
-
bug needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
debpalash/VoiceStudio#2624 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
router-for-me/CLIProxyAPI#6399 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
ReactionMechanismGenerator/ARC#1067 ·
メンテナーはふだん 1 日以内に返信