fix: eliminate N+1 queries in get_custom_commands (#26) #51
Open
cal
wants to merge 1 commits from
ai/major-domo-database-26 into next-release
pull from: ai/major-domo-database-26
merge into: cal:next-release
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/75-fix-n-1-query-pattern-in-standings-recalculation
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-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
|
ef9cd3d4a5 |
fix: eliminate N+1 queries in get_custom_commands (#26)
All checks were successful
Build Docker Image / build (pull_request) Successful in 2m1s
Expand the JOIN SELECT to include all creator fields (created_at, total_commands, active_commands) and build the CustomCommandCreatorModel directly from joined row data instead of issuing a separate SELECT per command. Reduces 26 queries to 2 for a full 25-command page. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> |