Fix N+1 query pattern in events findAll() #12

New Issue

kyle · 2026-02-08T06:44:19-08:00

kyle commented

2026-02-08 06:44:19 -08:00

Context: events.service.ts findAll() fetches all events, then loops through each to enrich with additional VIP data, creating an N+1 query pattern.

File: backend/src/events/events.service.ts (lines 88-105)

Suggested fix:

Use Prisma include with nested selects to fetch all needed VIP data in the initial query
Eliminate the per-event enrichment loop

Effort: Small

**Context:** `events.service.ts` `findAll()` fetches all events, then loops through each to enrich with additional VIP data, creating an N+1 query pattern. **File:** `backend/src/events/events.service.ts` (lines 88-105) **Suggested fix:** - Use Prisma `include` with nested selects to fetch all needed VIP data in the initial query - Eliminate the per-event enrichment loop **Effort:** Small

kyle referenced this issue

2026-02-08 06:44:30 -08:00

Code Efficiency #8

kyle referenced this issue from a commit

2026-02-08 07:07:27 -08:00

refactor: code efficiency improvements (Issues #9-13, #15, #17-20)

kyle commented

2026-02-08 07:09:57 -08:00

Refactored events to use Prisma includes instead of N+1 per-event enrichment loop.

kyle closed this issue

2026-02-08 07:09:57 -08:00

Sign in to join this conversation.