Consolidate redundant double-query in get_one_play #14

New Issue

cal · 2026-02-20T06:52:21Z

cal commented

2026-02-20 06:52:21 +00:00

`app/routers_v2/stratplays.py:1405-1412`. First calls `get_or_none` to check existence, discards result, then calls `get_by_id` again. Two round-trips for one record.

Priority: low

\`app/routers_v2/stratplays.py:1405-1412\`. First calls \`get_or_none\` to check existence, discards result, then calls \`get_by_id\` again. Two round-trips for one record. **Priority**: low

cal added the

performance

tech-debt

labels 2026-02-20 06:52:21 +00:00

cal added the

ai-working

label 2026-03-04 03:01:07 +00:00

cal referenced this issue from a commit

2026-03-04 03:02:51 +00:00

fix: consolidate redundant double-query in get_one_play (#14)

cal referenced a pull request that will close this issue

2026-03-04 03:02:59 +00:00

fix: consolidate redundant double-query in get_one_play (#14) #52

cal removed the

ai-working

label 2026-03-04 03:03:09 +00:00

cal commented

2026-03-04 03:03:12 +00:00

Fixed in PR #52: #52

Stored the get_or_none result instead of discarding it, then returned it directly — eliminating the redundant get_by_id call.

Fixed in PR #52: https://git.manticorum.com/cal/paper-dynasty-database/pulls/52 Stored the `get_or_none` result instead of discarding it, then returned it directly — eliminating the redundant `get_by_id` call.

cal added the

ai-pr-opened

label 2026-03-04 03:03:15 +00:00

cal referenced this issue from a commit

2026-03-05 03:36:03 +00:00

fix: consolidate redundant double-query in get_one_play (#14)

cal closed this issue

2026-03-05 03:36:27 +00:00

cal referenced this issue from a commit

2026-03-05 03:36:28 +00:00

Merge pull request 'fix: consolidate redundant double-query in get_one_play (#14)' (#52) from ai/paper-dynasty-database#14 into next-release

cal referenced this issue