perf(events): eliminate /events N+1 queries — 298 → 47 queries, ~1500ms → ~340ms by smarcet · Pull Request #549 · OpenStackweb/summit-api

fix(serializer): add last_edited timestamp to presentation cache key for automatic invalidation

smarcet

committed

chore(routes): enable server.timing.doctrine on getEvents for profiling

smarcet

committed

chore(profiling): expose per-phase request breakdown via Server-Timing header

smarcet

committed

fix(profiling): accurate DB time measurement via DBAL Driver Middleware

smarcet

committed

chore(profiling): log top-repeating SQL patterns to identify exact N+1 sources

smarcet

committed

fix(member): memoize belongsToGroup() per-instance to eliminate N+1 of 84 queries

smarcet

committed

fix(repository): preload PresentationSpeaker + Member in single DQL query

smarcet

committed

fix(repository): include root alias 'a' in speaker+member preload SELECT

smarcet

committed

chore(debug): instrument preload to confirm what it loads

smarcet

committed

chore(debug): capture Member query params to identify which Members are being loaded

smarcet

committed

fix(debug): strip backticks before matching Member table name

smarcet

committed

fix(auth): memoize getCurrentUser() — eliminates ~98 redundant Member SELECTs per request

smarcet

committed

fix(repository): batch-preload SummitSelectedPresentation, eliminates 20 queries

smarcet

committed

chore(debug): instrument SummitSelectedPresentation preload to find why it has no effect

smarcet

committed

fix(speaker): pre-populate getPresentationAssignmentOrder cache from batch-loaded assignments

smarcet

committed

fix(repository): preload location (FETCH JOIN), tags and materials in /events hydration

smarcet

committed

fix(repository): preload sponsors batch + fetch-join PresentationCategory

smarcet

committed

chore(cleanup): remove diagnostic logging used to profile the /events N+1 hunt

smarcet

committed

docs(adr): document /events N+1 elimination work

smarcet

committed

fix(events): correct SummitSelectedPresentation preload DQL and cache reset on auth context change

smarcet

committed

fix(events): low-severity findings + unit tests from PR #549 review

smarcet

committed

fix(trait): restore optional afterQuery hook in _getAll

smarcet

committed

fix(timing): replace Session with request attributes for sub-phase timings

smarcet

committed

fix(timing): use import instead of fully qualified class for QueryTimingCollector

smarcet

committed

fix(auth): track side-effects independently in getCurrentUser cache

smarcet

committed

fix(presentation): reset memoizedSelectionStatus on all state-mutating methods

smarcet

committed

fix(cache): bump last_edited on association-only mutations to bust PresentationSerializer cache

smarcet

committed

fix(cache): invalidate presentation cache on speaker reorder and harden getCurrentUser

smarcet

committed

chore(config): commit message rules updated

smarcet

committed

fix(presentation): clear preload caches on plan/material mutations

smarcet

committed

fix(presentation): clear preloadedSessionSelections in setSelectedPresentations and setCategory

smarcet

committed

fix(auth): restore cachedCurrentUser after syncMemberFields transaction

smarcet

committed

chore(routes): fix middleware order

smarcet

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

perf(events): eliminate /events N+1 queries — 298 → 47 queries, ~1500ms → ~340ms#549

perf(events): eliminate /events N+1 queries — 298 → 47 queries, ~1500ms → ~340ms#549
smarcet wants to merge 33 commits into
mainfrom
hotfix/cache-optimizations

Commits on Jun 12, 2026

fix(serializer): add last_edited timestamp to presentation cache key for automatic invalidation

chore(routes): enable server.timing.doctrine on getEvents for profiling

chore(profiling): expose per-phase request breakdown via Server-Timing header

fix(profiling): accurate DB time measurement via DBAL Driver Middleware

chore(profiling): log top-repeating SQL patterns to identify exact N+1 sources

fix(member): memoize belongsToGroup() per-instance to eliminate N+1 of 84 queries

fix(repository): preload PresentationSpeaker + Member in single DQL query

fix(repository): include root alias 'a' in speaker+member preload SELECT

chore(debug): instrument preload to confirm what it loads

chore(debug): capture Member query params to identify which Members are being loaded

fix(debug): strip backticks before matching Member table name

fix(auth): memoize getCurrentUser() — eliminates ~98 redundant Member SELECTs per request

fix(repository): batch-preload SummitSelectedPresentation, eliminates 20 queries

chore(debug): instrument SummitSelectedPresentation preload to find why it has no effect

fix(speaker): pre-populate getPresentationAssignmentOrder cache from batch-loaded assignments

fix(repository): preload location (FETCH JOIN), tags and materials in /events hydration

fix(repository): preload sponsors batch + fetch-join PresentationCategory

chore(cleanup): remove diagnostic logging used to profile the /events N+1 hunt

docs(adr): document /events N+1 elimination work

fix(events): correct SummitSelectedPresentation preload DQL and cache reset on auth context change

fix(events): low-severity findings + unit tests from PR #549 review

fix(trait): restore optional afterQuery hook in _getAll

fix(timing): replace Session with request attributes for sub-phase timings

fix(timing): use import instead of fully qualified class for QueryTimingCollector

fix(auth): track side-effects independently in getCurrentUser cache

fix(presentation): reset memoizedSelectionStatus on all state-mutating methods

fix(cache): bump last_edited on association-only mutations to bust PresentationSerializer cache

fix(cache): invalidate presentation cache on speaker reorder and harden getCurrentUser

chore(config): commit message rules updated

fix(presentation): clear preload caches on plan/material mutations

fix(presentation): clear preloadedSessionSelections in setSelectedPresentations and setCategory

fix(auth): restore cachedCurrentUser after syncMemberFields transaction

chore(routes): fix middleware order