[Performance] N+1-Abfragen beim Laden eines Wochenplans #49

New issue

Closed

opened 2026-09-09 20:17:01 +00:00 by Aceless · 1 comment

Aceless commented

2026-09-09 20:17:01 +00:00

Owner

Problem

Data/MealPlanRepository.LoadEntries laedt fuer jeden Eintrag das komplette Rezept:

foreach (var e in entries)
{
    ...
    if (entry.RecipeId.HasValue)
        entry.Recipe = await _recipeRepository.GetRecipeById(entry.RecipeId.Value);
}

GetRecipeById fuehrt sechs Statements aus (Rezept, Zutaten, Schritte, Tags, SuitableFor, Bilder).

Rechnung

Eintraege pro Woche	Statements
7	~44
21	~128

Dazu kommt: Der Wochenplan liefert am Ende nur RecipeListItemDto, Zutaten und Zubereitungsschritte werden also geladen und dann verworfen.

Fix

Alle Rezept-Ids der Eintraege sammeln und in einem QueryMultiple mit WHERE recipe_id = ANY(@Ids) laden, danach im Speicher zuordnen (so wie es GetRecipesInSpace bereits macht).
Eine schlanke Lademethode fuer Listendarstellungen ergaenzen, die Zutaten und Schritte gar nicht erst abfragt.
Denselben Ansatz in AddEntry und UpdateEntry verwenden.

## Problem `Data/MealPlanRepository.LoadEntries` laedt fuer jeden Eintrag das komplette Rezept: ```csharp foreach (var e in entries) { ... if (entry.RecipeId.HasValue) entry.Recipe = await _recipeRepository.GetRecipeById(entry.RecipeId.Value); } ``` `GetRecipeById` fuehrt sechs Statements aus (Rezept, Zutaten, Schritte, Tags, SuitableFor, Bilder). ## Rechnung | Eintraege pro Woche | Statements | |---|---| | 7 | ~44 | | 21 | ~128 | Dazu kommt: Der Wochenplan liefert am Ende nur `RecipeListItemDto`, Zutaten und Zubereitungsschritte werden also geladen und dann verworfen. ## Fix - Alle Rezept-Ids der Eintraege sammeln und in **einem** `QueryMultiple` mit `WHERE recipe_id = ANY(@Ids)` laden, danach im Speicher zuordnen (so wie es `GetRecipesInSpace` bereits macht). - Eine schlanke Lademethode fuer Listendarstellungen ergaenzen, die Zutaten und Schritte gar nicht erst abfragt. - Denselben Ansatz in `AddEntry` und `UpdateEntry` verwenden.

Aceless added the

performance

label

2026-09-09 20:17:01 +00:00

Aceless referenced this issue from a commit

2026-09-09 21:18:12 +00:00

Essensplan: Zugriff auf fremde Plaene, Race Conditions, N+1

Aceless commented

2026-09-09 21:18:51 +00:00

Author

Owner

Erledigt.

LoadEntriesAsync sammelt die Rezept-Ids aller Einträge und lädt sie über die neue Methode GetRecipeSummariesAsync mit WHERE id = ANY(@Ids). Zutaten und Zubereitungsschritte werden dabei gar nicht erst geholt, der Wochenplan zeigt ohnehin nur RecipeListItemDto.

Gemessen mit log_statement = all auf einer Testdatenbank, Wochenplan mit sieben Einträgen und sieben verschiedenen Rezepten:

	Statements pro Abruf
vorher	45
jetzt	7

Die sieben sind Rollenprüfung, Wochenplan, Einträge und je eine Abfrage für Rezepte, Tags, Mealtypes und Bilder. Die Zahl bleibt konstant, unabhängig von der Anzahl der Einträge. Bei einer vollen Woche mit 21 Einträgen waren es vorher 129.

GetRecipesInSpace nutzt jetzt Joins statt der dreifach wiederholten Unterabfrage.

Branch backend/issue-fixes, Commits 77327fc, 3957404.

Erledigt. `LoadEntriesAsync` sammelt die Rezept-Ids aller Einträge und lädt sie über die neue Methode `GetRecipeSummariesAsync` mit `WHERE id = ANY(@Ids)`. Zutaten und Zubereitungsschritte werden dabei gar nicht erst geholt, der Wochenplan zeigt ohnehin nur `RecipeListItemDto`. **Gemessen** mit `log_statement = all` auf einer Testdatenbank, Wochenplan mit sieben Einträgen und sieben verschiedenen Rezepten: | | Statements pro Abruf | |---|---| | vorher | 45 | | jetzt | 7 | Die sieben sind Rollenprüfung, Wochenplan, Einträge und je eine Abfrage für Rezepte, Tags, Mealtypes und Bilder. Die Zahl bleibt konstant, unabhängig von der Anzahl der Einträge. Bei einer vollen Woche mit 21 Einträgen waren es vorher 129. `GetRecipesInSpace` nutzt jetzt Joins statt der dreifach wiederholten Unterabfrage. --- Branch `backend/issue-fixes`, Commits 77327fc, 3957404.

Aceless closed this issue

2026-09-09 21:18:51 +00:00