Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

issue 10727: optimize performance for dubbo protocol #10728

Merged
merged 8 commits into from
Oct 14, 2022
Merged

issue 10727: optimize performance for dubbo protocol #10728

merged 8 commits into from
Oct 14, 2022

Conversation

icodening
Copy link
Contributor

与Triple协议性能优化类似，同样使用了缓冲队列来大幅度提高dubbo协议的性能问题
ref #10727 #10558

@icodening icodening changed the base branch from 3.1 to 3.2 October 13, 2022 06:31
EarthChen
EarthChen approved these changes Oct 14, 2022
View reviewed changes
Copy link
Member

@EarthChen EarthChen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AlbumenJ AlbumenJ merged commit 348e316 into apache:3.2 Oct 14, 2022
@icodening
Copy link
Contributor Author

环境信息：

  • 6C 16G
  • mac OS 10.15.7
  • jdk8
优化后：dubbo/fastjson2
Benchmark           Mode  Cnt      Score       Error  Units
Client.createUser  thrpt    3  78492.180 ± 52746.055  ops/s
Client.existUser   thrpt    3  93525.607 ± 36409.882  ops/s
Client.getUser     thrpt    3  83727.914 ±  4138.831  ops/s
Client.listUser    thrpt    3  32823.162 ± 26790.908  ops/s

优化前：3.1.1 dubbo/fastjson2
Benchmark           Mode  Cnt      Score       Error  Units
Client.createUser  thrpt    3  62864.959 ± 18826.824  ops/s
Client.existUser   thrpt    3  72422.411 ± 39122.440  ops/s
Client.getUser     thrpt    3  62999.750 ± 15666.732  ops/s
Client.listUser    thrpt    3  33994.761 ± 13043.967  ops/s

@icodening icodening mentioned this pull request Nov 10, 2022
Open
@icodening icodening deleted the 10727 branch December 26, 2022 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlbumenJ AlbumenJ AlbumenJ requested changes

@EarthChen EarthChen EarthChen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@icodening @AlbumenJ @EarthChen