Commit 47408b2
committed
fix(cases): 第三轮架构师审核修复 6 处技术错误
第三轮深入审核发现 28 个问题,本轮修复 6 个确凿的'会让读者跑出非预期结果'的技术错误,剩余 22 个偏文档风格/版本细节问题留待后续。
## 严重问题(5)
**#R1 案例 84 文档 bad.sql 是占位符**
- 原 bad.sql 第 52-53 行:'SELECT ... FROM t1, t2 LIMIT 50000'
- 't1, t2' 是占位符不是真实表,直接复制运行报 Table doesn't exist
- 替换为指向 sql/cases/84-tidb-statistics/bad.sql 真实可执行 SQL
**#R2 案例 88 bad/good WHERE 条件不一致**
- bad.sql 删 status=0
- good.sql 改删 status=1
- 条件不一致破坏对比意义
- good 改为同条件 status=0 LIMIT 1000
**#R3 案例 90 tidb_mem_quota_query 数字矛盾**
- 第 13 行:'HashAgg 吃了将近 2GB,tidb_mem_quota_query 设置为 1GB'
- bad.sql 第 43 行:'SET SESSION tidb_mem_quota_query = 104857600 (100MB)'
- 2GB/1GB 与 100MB 实际设置直接矛盾
- 改为:'HashAgg 吃了将近 200MB,tidb_mem_quota_query 设置为 100MB'
**#R4 案例 91 Index Join 性能计算错误**
- 原描述:'90万次比较远小于 10万行全扫 + 构建哈希'
- 在 TiDB 分布式场景下,Index Join 的 10万次 IndexRangeScan 每次是 RPC 调用(~0.5-1ms),总开销约 50 秒
- Hash Join 反而远优于 Index Join(一次全表扫 ~100ms)
- 这是案例主题的关键事实错误
- 改为:'10万次 RPC 是真正瓶颈,Hash Join 远优'
**#R5 案例 88 MySQL 变量名在 TiDB 中不生效(3处)**
- max_execution_time(MySQL 8.0 变量)→ tidb_max_execution_time_ms(TiDB 7.x 变量)
- 涉及 L145 SET、L165 文档说明、L222 推荐表
- TiDB 不识别 max_execution_time,配置无效
## 中等问题(1)
**#R6 案例 99 innodb_lock_wait_timeout 默认值错误**
- 原表:'MySQL 默认 50s | TiDB 默认 50s'
- 实际 TiDB 7.x 默认 3s(不是 50s)
- v6.x 及以下才是 50s
- 改为:'TiDB 7.x 默认 3s;v6.x 及以下默认 50s'
**#R7 案例 65 next-key lock 描述补充**
- '锁定行数 ~20,031' 是记录锁命中数,RR 下还会加间隙锁
- 补充:'实际含间隙锁,锁范围更大'
## 验证
- npm run docs:build 成功
- npm run docs:pdf 生成 55.0 MB / 748 页 / 113 书签
- 6 个文件修改,+18 / -14 行1 parent 6719510 commit 47408b2
7 files changed
Lines changed: 18 additions & 14 deletions
File tree
- docs
- cases
- tidb
- transaction
- public
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
52 | | - | |
53 | | - | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
54 | 54 | | |
55 | 55 | | |
56 | 56 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
142 | 142 | | |
143 | 143 | | |
144 | 144 | | |
145 | | - | |
| 145 | + | |
146 | 146 | | |
147 | 147 | | |
148 | 148 | | |
149 | 149 | | |
150 | 150 | | |
151 | | - | |
152 | | - | |
| 151 | + | |
| 152 | + | |
153 | 153 | | |
154 | 154 | | |
155 | 155 | | |
| |||
162 | 162 | | |
163 | 163 | | |
164 | 164 | | |
165 | | - | |
| 165 | + | |
166 | 166 | | |
167 | 167 | | |
168 | 168 | | |
| |||
219 | 219 | | |
220 | 220 | | |
221 | 221 | | |
222 | | - | |
| 222 | + | |
223 | 223 | | |
224 | 224 | | |
225 | 225 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
| 13 | + | |
14 | 14 | | |
15 | | - | |
| 15 | + | |
16 | 16 | | |
17 | 17 | | |
18 | 18 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
145 | 145 | | |
146 | 146 | | |
147 | 147 | | |
148 | | - | |
149 | | - | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
150 | 154 | | |
151 | 155 | | |
152 | 156 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
75 | 75 | | |
76 | 76 | | |
77 | 77 | | |
78 | | - | |
| 78 | + | |
79 | 79 | | |
80 | 80 | | |
81 | 81 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
168 | 168 | | |
169 | 169 | | |
170 | 170 | | |
171 | | - | |
| 171 | + | |
172 | 172 | | |
173 | 173 | | |
174 | 174 | | |
| |||
Binary file not shown.
0 commit comments