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

bbdc74d

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

smarcet

committed

cf19f31

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

smarcet

committed

5a206b2

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

smarcet

committed

3d0264d

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

smarcet

committed

7968d40

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

smarcet

committed

4d78aa1

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

smarcet

committed

d3e00ea

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

smarcet

committed

04d63a7

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

smarcet

committed

586f202

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

smarcet

committed

b1939fa

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

smarcet

committed

736e7b3

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

smarcet

committed

1107c5b

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

smarcet

committed

b4ec71e

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

smarcet

committed

176c91d

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

smarcet

committed

af997fd

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

smarcet

committed

bed486f

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

smarcet

committed

05e300e

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

smarcet

committed

179b915

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

smarcet

committed

2073af3

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

smarcet

committed

60e814a

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

smarcet

committed

722556a

fix(trait): restore optional afterQuery hook in _getAll

smarcet

committed

e774876

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

smarcet

committed

746512a

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

smarcet

committed

d086f11

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

smarcet

committed

8d5b833

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

smarcet

committed

f6800e7

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

smarcet

committed

a295d99

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

smarcet

committed

9d095d5

chore(config): commit message rules updated

smarcet

committed

ba81308

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

smarcet

committed

4ffcc84

fix(presentation): clear preloadedSessionSelections in setSelectedPresentations and setCategory

smarcet

committed

588c3c6

fix(auth): restore cachedCurrentUser after syncMemberFields transaction

smarcet

committed

094faa6

chore(routes): fix middleware order

smarcet

committed

926ee8a

Navigation Menu

Uh oh!

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 merged 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