lzf_0626
6b2e2edcda
## 问题
`trade_service._next_id` 用 `SELECT MAX(id)+1` 发主键。两个事务读到同一个 MAX、
算出同一个 id，后写的那笔 `flush()` 撞 `Duplicate entry ... for key 'PRIMARY'`
-> 该客户下单直接 **500**。`submit_order` 一次要发 **3 个 id**
（订单 / 成交 / 资金流水），冲突面是单表的三倍。
## 这是"修正偏差"，不是"改基线"
`docs/00-新数据库基线设计.md` 第 41 行：
| 主键 | 统一 `BIGINT UNSIGNED AUTO_INCREMENT`，业务编号另设唯一键 |
**基线本来就要求 AUTO_INCREMENT**，是生成的 DDL 漏了 —— `_next_id` 自己的
docstring 也写着"与 docs/00 设计稿存在偏差"。所以本迁移**不违反** AGENTS.md
规则 4（禁止改类型/可空性/业务含义）：类型仍是 `BIGINT UNSIGNED`、仍是 `NOT NULL`、
`id` 的业务含义不变，只是补回一个列属性；已有行 id 不变，显式给 id 依然合法。
## 迁移 `20260914_baseline_auto_increment`
- **15 张表**恢复 AUTO_INCREMENT（硬编码表名 —— 迁移必须确定性，动态查
`information_schema` 会让同一份迁移在不同环境产生不同结果）。
- **有意排除 3 张**（`EXCLUDED_BECAUSE_FOREIGN_KEY`）：`fin_product`（被 10 张
`advisor_product_*` 引用）、`fin_risk_assessment`、`sys_user`（被 19+ 张引用）。
MySQL 拒绝 `MODIFY` 被外键引用的列：
(1833, "Cannot change column 'id': used in a foreign key constraint ...")
改它们必须先 DROP FOREIGN KEY -> MODIFY -> 重建外键，那是另一件事（涉及 30+ 个
外键的重建与一致性验证），不该塞进这条"恢复基线属性"的迁移。且这三张表**写入
频率很低、没有任何代码用 `SELECT MAX(id)+1` 给它们发号** —— 排除它们不影响
本迁移的目标。
- `downgrade()` 可回滚（只是去掉属性、不丢数据），但注释里写明：**回滚会把 P0-2
的并发冲突带回来**。
⚠️ 迁移执行中踩到过"部分生效"：MySQL DDL 非事务性，第一次跑到 `fin_product`
才报错，**前 7 张已经改完**。修正列表后重跑即收敛（对已是 AUTO_INCREMENT 的列
再 `MODIFY` 是无害的）。这一点也说明**迁移必须逐表可重入**。
## 代码
`trade_service.py` 删除 `_next_id` 方法及 4 处调用（`FundSimOrder` /
`FundTransaction` / `FundCashLedger` / `FundHolding`），改由 InnoDB 分配；
顺带清掉因此不再使用的 `Any` 与 `func` import（全仓 grep 确认它们只服务于
`_next_id`）。测试对 `_next_id` 零依赖（已 grep 确认）。
`test_advisor_migration_contract.py` 里那个"钉住末端版本"的断言按它自己的注释
要求同步更新到新 head。
## 实测
- `alembic upgrade head` -> `current = 20260914_baseline_auto_increment`，
复核状态：**15 张已生效、3 张按设计排除**
- **并发下单实测**（2 个客户 × 3 笔 = 6 笔真并发；刻意用**不同客户**，
因为 P0-3 的行锁已经把同一客户串行化了，不同客户才会真正并发进入发号路径）：
成功 6 / 主键冲突 0 / 其它失败 0
=> P0-2 已解决
- `pytest tests/unit tests/contract` -> **1427 passed, 2 skipped, 2 failed**
（2 个既有失败与本次无关）
- `ruff check` -> All checks passed
70 lines
3.1 KiB
Python
70 lines
3.1 KiB
Python
from __future__ import annotations
|
|
|
|
import re
|
|
from pathlib import Path
|
|
|
|
from alembic.config import Config
|
|
from alembic.script import ScriptDirectory
|
|
|
|
ROOT = Path(__file__).resolve().parents[2]
|
|
VERSIONS = ROOT / "alembic" / "versions"
|
|
BASELINE = ROOT / "alembic" / "baseline_generated.sql"
|
|
|
|
ADVISOR_FILES = (
|
|
"20260910_advisor_investment_goal.py",
|
|
"20260910_advisor_product_industry_exposure.py",
|
|
"20260910_advisor_portfolio_projection_checkpoint.py",
|
|
"20260910_advisor_product_metric_snapshot.py",
|
|
"20260910_advisor_product_asset_classification.py",
|
|
"20260910_advisor_product_reference_snapshot.py",
|
|
"20260910_advisor_offsite_fund_reference.py",
|
|
"20260910_advisor_product_price_history.py",
|
|
"20260910_advisor_product_governance_reference.py",
|
|
"20260910_advisor_governance_monitor_and_quotes.py",
|
|
"20260910_advisor_market_quote_resilience.py",
|
|
"20260910_advisor_data_quality_backtest.py",
|
|
"20260911_advisor_goal_conversation.py",
|
|
"20260911_advisor_profile_tag_governance.py",
|
|
)
|
|
|
|
|
|
def created_tables(path: Path) -> set[str]:
|
|
content = path.read_text(encoding="utf-8")
|
|
return set(re.findall(r"CREATE TABLE(?: IF NOT EXISTS)?\s+`?([A-Za-z0-9_]+)`?", content))
|
|
|
|
|
|
def test_advisor_migrations_form_one_chain_from_qyqy_head() -> None:
|
|
script = ScriptDirectory.from_config(Config(str(ROOT / "alembic.ini")))
|
|
assert len(script.get_heads()) == 1
|
|
# 核心断言是上面那句"链收敛到一个 head"；下面钉住当前末端版本，便于发现迁移被误删或分叉。
|
|
# ⚠️ **新增迁移后要同步更新这个值**。2026-09-13 追加了
|
|
# `20260913_market_price_change_pct`（给 `fin_market_price` 补 `change_pct`）；
|
|
# 2026-09-14 追加了 `20260914_baseline_auto_increment`（给 15 张表恢复基线要求的
|
|
# `AUTO_INCREMENT`，消除 `_next_id` 的并发主键冲突）。
|
|
assert script.get_heads()[0] == "20260914_baseline_auto_increment"
|
|
|
|
first = (VERSIONS / ADVISOR_FILES[0]).read_text(encoding="utf-8")
|
|
assert 'down_revision = "20260910_drop_review_separation"' in first
|
|
for previous, current in zip(ADVISOR_FILES, ADVISOR_FILES[1:], strict=False):
|
|
previous_content = (VERSIONS / previous).read_text(encoding="utf-8")
|
|
current_content = (VERSIONS / current).read_text(encoding="utf-8")
|
|
previous_revision = re.search(r'revision = "([^"]+)"', previous_content)
|
|
assert previous_revision is not None
|
|
assert f'down_revision = "{previous_revision.group(1)}"' in current_content
|
|
|
|
|
|
def test_advisor_migrations_only_create_additive_tables() -> None:
|
|
baseline_tables = created_tables(BASELINE)
|
|
advisor_tables: set[str] = set()
|
|
for filename in ADVISOR_FILES:
|
|
content = (VERSIONS / filename).read_text(encoding="utf-8")
|
|
assert "ALTER TABLE" not in content
|
|
assert "DROP TABLE" not in content
|
|
advisor_tables.update(created_tables(VERSIONS / filename))
|
|
|
|
assert advisor_tables
|
|
assert not advisor_tables & baseline_tables
|
|
assert "fin_sim_order" not in advisor_tables
|
|
assert "fin_transaction" not in advisor_tables
|
|
assert "fin_holding" not in advisor_tables
|