fix: batch standings updates to eliminate N+1 queries in recalculate (#75) #93
Open
Claude
wants to merge 1 commits from
issue/75-fix-n-1-query-pattern-in-standings-recalculation into main
pull from: issue/75-fix-n-1-query-pattern-in-standings-recalculation
merge into: cal:main
cal:main
cal:issue/101-fieldingstats-get-totalstats-total-count-overwritt
cal:issue/71-refactor-manual-db-close-calls-to-middleware-based
cal:fix/remove-hardcoded-webhook
cal:issue/81-add-migration-tracking-system
cal:issue/74-add-missing-indexes-on-foreign-key-columns-in-high
cal:issue/69-boolean-fields-compared-as-integers-sqlite-pattern
cal:issue/77-replace-row-by-row-delete-with-bulk-delete-in-care
cal:issue/82-align-python-version-between-docker-image-3-11-and
cal:issue/76-replace-deprecated-pydantic-dict-with-model-dump
cal:issue/70-remove-sqlite-fallback-code-from-db-engine-py
cal:issue/78-fix-discord-id-type-mismatch-between-model-charfie
cal:issue/80-disable-autoconnect-and-set-pool-timeout-on-pooled
cal:issue/73-add-type-annotations-to-untyped-query-parameters
cal:issue/79-stop-logging-raw-auth-tokens-in-warning-messages
cal:ai/major-domo-database-66
cal:ai/major-domo-database-65
cal:next-release
cal:ai/major-domo-database-38
cal:ai/major-domo-database-27
cal:ai/major-domo-database-19
cal:ai/major-domo-database-20
cal:ai/major-domo-database-22
cal:ai/major-domo-database-26
cal:ai/major-domo-database-31
cal:ai/major-domo-database-35
cal:feat/sbaplayer-id-plays-filter
cal:chore/optimize-claude-md
1 Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
Cal Corum
|
d3b9355f26 |
fix: batch standings updates to eliminate N+1 queries in recalculate (#75)
All checks were successful
Build Docker Image / build (pull_request) Successful in 2m54s
Replace per-game update_standings() calls with pre-fetched dicts and in-memory accumulation, then a single bulk_update at the end. Reduces ~1,100+ queries for a full season to ~5 queries. Closes #75 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> |