Skip to content

去掉去性能没有影响的代码#472

Merged
guonaihong merged 2 commits intomasterfrom
del-code
May 8, 2025
Merged

去掉去性能没有影响的代码#472
guonaihong merged 2 commits intomasterfrom
del-code

Conversation

@guonaihong
Copy link
Collaborator

修改之前

A 6900hx机器数据
sec: 476, recv-count: 260486422, send-count:260486407 recv-tps: 547240, send-tps: 547240

B 6900hx机器数据
sec: 150, recv-count: 86958496, send-count:86958482 recv-tps: 579723, send-tps: 579723

修改之后
A 6900hx机器数据
sec: 283, recv-count: 153606508, send-count:153606492 recv-tps: 542779, send-tps: 542779

B 6900hx机器数据
sec: 150, recv-count: 87004344, send-count:87004336 recv-tps: 580028, send-tps: 580028

A机器散热不太好,空负载第一次跑数据好点,第二次数据低点。B机器散热好,可以发现两次数据接近

guonaihong added 2 commits May 9, 2025 00:16
修改之前

A 6900hx机器数据
sec: 476, recv-count: 260486422, send-count:260486407 recv-tps: 547240, send-tps: 547240

B 6900hx机器数据
sec: 150, recv-count: 86958496, send-count:86958482 recv-tps: 579723, send-tps: 579723

修改之后
A 6900hx机器数据
sec: 283, recv-count: 153606508, send-count:153606492 recv-tps: 542779, send-tps: 542779

B 6900hx机器数据
sec: 150, recv-count: 87004344, send-count:87004336 recv-tps: 580028, send-tps: 580028

A机器散热不太好,空负载第一次跑数据好点,第二次数据低点。B机器散热好,可以发现两次数据接近
@guonaihong guonaihong merged commit ce36c51 into master May 8, 2025
7 checks passed
@lesismal
Copy link
Owner

lesismal commented May 8, 2025

之前是参考字节的文章里的实现,据说有性能提升;
实际测试和使用中对比下来,没有发现有效提升,所以恢复到简单实现。

Reference/Related:
https://www.cloudwego.io/blog/2021/09/23/performance-optimization-on-kitex/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants