Zhenghao Zhang (@zhenghaoz) 在 氛围编程自己都不看代码的么 PR 上来就删除我项目的并发能力😭 中发帖
晚上起夜发现开源项目有个review没看,仔细一看更睡不着了
BUG说明
为了加速MySQL(或者PG)的全表扫描速度,将第一个主键(从其他表获得)按照范围进行分组,每个分组之间进行并发扫描。然而,某些特定collation和Go的排序方式不同,导致数据漏扫或者多扫。
人类解决方案
实际上解决方案不少:
建表语句(项目内)强制collation,然后线上表变更collation
复杂点做法可以根据实际collation划分第一个主键
氛围编程的解法
直接删除了并发…
[FireShot Capture 013 - Fix panic and data race in parallel feedback loading by quarkness ·_ - github.com]
就算不并发了,把分组的逻辑删除掉也行,怎么还把原来并发用的逻辑留着…
用氛围编程贡献代码...