AURORA-1831: гонка technical_number при параллельном POST /projects #229

aep · 2026-10-28T17:14:56+03:00

aep commented

2026-10-28 17:14:56 +03:00

Closes #207

Jira: AURORA-1831

Что сделано

Миграция 0050: v3.add_project (копия версии из 0018) с pg_advisory_xact_lock(hashtext('v3.project.technical_number'), p_org_unit_id) перед вычислением MAX. Решение опирается на READ COMMITTED (комментарий в SQL).
Счётчик > 99: RAISE EXCEPTION с понятным текстом вместо тихой обрезки lpad. Хэндлер отдаёт 400 с этим текстом (новых маппингов нет). Формат номера не менялся.
Не в этой задаче: повтор номера после удаления и обрезка org_unit_id >= 10000 (AURORA-1919).

Тесты

tests/integration/test_project_technical_number_concurrency.py: гонка на двух соединениях (ожидание блокировки через pg_stat_activity, уборка в finally; до правки unique violation) и переполнение счётчика (вставка номера с 99 в транзакции с откатом).
Полный прогон на PG16: 202 passed, 4 skipped, 3 failed (исходные падения test, не связаны).

Проверить руками

Два параллельных POST /api/v1/projects на одну оргединицу: оба 200, номера различаются на 1 в последних двух цифрах.
Номер 0050 могут занять другие PR: при rebase проверить down_revision.

Замер до/после

Будет добавлен после ночного прогона на стенде perf/: параллельное создание проектов в одной оргединице, ожидаются 0 ответов 409.

Rulings I made

нет

🤖 Generated with Claude Code

Closes #207 Jira: AURORA-1831 ## Что сделано - Миграция 0050: v3.add_project (копия версии из 0018) с pg_advisory_xact_lock(hashtext('v3.project.technical_number'), p_org_unit_id) перед вычислением MAX. Решение опирается на READ COMMITTED (комментарий в SQL). - Счётчик > 99: RAISE EXCEPTION с понятным текстом вместо тихой обрезки lpad. Хэндлер отдаёт 400 с этим текстом (новых маппингов нет). Формат номера не менялся. - Не в этой задаче: повтор номера после удаления и обрезка org_unit_id >= 10000 (AURORA-1919). ## Тесты - tests/integration/test_project_technical_number_concurrency.py: гонка на двух соединениях (ожидание блокировки через pg_stat_activity, уборка в finally; до правки unique violation) и переполнение счётчика (вставка номера с 99 в транзакции с откатом). - Полный прогон на PG16: 202 passed, 4 skipped, 3 failed (исходные падения test, не связаны). ## Проверить руками - Два параллельных POST /api/v1/projects на одну оргединицу: оба 200, номера различаются на 1 в последних двух цифрах. - Номер 0050 могут занять другие PR: при rebase проверить down_revision. ## Замер до/после Будет добавлен после ночного прогона на стенде perf/: параллельное создание проектов в одной оргединице, ожидаются 0 ответов 409. ## Rulings I made нет 🤖 Generated with [Claude Code](https://claude.com/claude-code)

aep added 2 commits 2026-10-28 17:15:01 +03:00

fix(projects): advisory lock и ошибка переполнения счётчика technical_number в add_project (AURORA-1831) 95b74bc5e5

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

test(projects): гонка technical_number и переполнение счётчика (AURORA-1831) 663f45ee04

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

aep commented

2026-10-28 17:16:04 +03:00

Проверка на PostgreSQL 15.12 (контроллер, 28.10): ветка + коммиты AURORA-1922 (PR #221, без них test на 15 не накатывается), пустая БД, --init_db — миграция 0050 проходит; полный pytest: 204 passed, 4 skipped, 3 failed (исходные падения test). Новых падений нет.

tsygankoviva approved these changes 2026-10-30 18:20:40 +03:00

aep merged commit 37bb563f5d into test

2026-11-01 09:45:36 +03:00

aep referenced this issue from a commit

2026-11-01 09:45:37 +03:00

Merge pull request 'AURORA-1831: гонка technical_number при параллельном POST /projects' (#229) from feat/207-aurora-1831-project-tn-race into test

aep referenced this issue from a commit

2026-11-01 09:46:30 +03:00

fix(alembic): перенумеровать 0050→0051 во избежание конфликта с #229

aep referenced this issue from a commit

2026-11-01 11:03:49 +03:00

fix(alembic): перенумеровать 0050→0053 во избежание конфликта с #229/#231/#244