今天在写 Kitex 框架关于负载均衡算法的源码解析,在跑 Benchmark 的时候突然发现我之前贡献的一个负载均衡算法性能有很大的倒退,折腾了好久都没明白,甚至把我当时本地的几个 commit 都签出测试还是一样
排查来排查去,感觉可能是依赖的 fastrand 有问题,框架里依赖的是字节的 fastrand
import "github.com/bytedance/gopkg/lang/fastrand"
但是我过去一看,这个 fastrand 已经两年没动过了,但是这个 fastrand 又依赖 runtimex
包里的 Fastrand()
,这个包更是三年没动过了(
我就去看这个 runtimex
是什么东西
package runtimex
import (
_ "unsafe" // for linkname
)
//go:linkname Fastrand runtime.fastrand
func Fastrand() uint32
啊,原来是直接搬来的 runtime
里的 fastrand
,我一想,难道是 Go 的版本问题吗?
于是我写了个 Benchmark ,再把 Go 从 1.22 调到 1.21 ,卧槽,的确是这个原因
于是我在 Kitex 里也改成了 1.21 ,的确和当时一模一样了
我去 issue 和 PR 里找相关的内容,但是没找到,但这个 fastrand 肯定很多人都在用吧,怎么我成了第一个发现的了
于是我自己交了一个 PR bytedance/gopkg#206 ,但这个仓库好像已经不怎么维护了,一堆 issue 和 PR 挂着()
原因就是 1.22 的更新导致的,找到了一篇文章 https://zhuanlan.zhihu.com/p/673906980
1
lesismal 293 天前 1
kitex-benchmark 后来按默认的脚本也跑不起来,不知道现在能不能跑了,kitex 代码太复杂我表示“看不懂”,所以不玩了,还是玩我自己的吧
连同 grpc ,都挺难用的 |
2
totoro52 293 天前
有可能在维护这个仓库的人被开源节流了
|
3
sealingpp 293 天前
牛的
|
4
thevita 293 天前
然后呢,是不是可以直接用 math/rand 了?
|
6
dyllen 292 天前
1.22 版本不是上了个 v2 版本的 rand 吗?估计改了不少内容。
|
7
xiaocaiji111 287 天前
go1.22 中有不少跟以前不兼容的地方,看官方的 ISSUE 就知道,破坏了不少兼容性
|