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

DataSenderFactory中的DataSenderChecker对于按百分比选择连接，使用更合理的分散算法与策略 #2

Closed
wu-sheng opened this issue Nov 26, 2015 · 4 comments
Closed

DataSenderFactory中的DataSenderChecker对于按百分比选择连接，使用更合理的分散算法与策略 #2

wu-sheng opened this issue Nov 26, 2015 · 4 comments
Labels
bug Something isn't working and you are sure it's a bug!
Milestone
1.0-alpha1

Comments

@wu-sheng
Copy link
Member

百分比会默认选择靠前的连接，会造成集群中的服务器连接数量不均衡。
服务器列表靠前的服务器，连接数过大。
@wu-sheng wu-sheng added this to the 1.0 milestone Nov 26, 2015
@wu-sheng wu-sheng added the bug Something isn't working and you are sure it's a bug! label Nov 26, 2015
@ascrutae
Copy link
Member

ascrutae commented Dec 1, 2015

在DataSenderChecker初始化的时候添加，添加随机获取服务方法。

@ascrutae ascrutae closed this as completed Dec 1, 2015
@wu-sheng
Copy link
Member Author

wu-sheng commented Dec 4, 2015

DataSenderChecker只在提取的时候随机，未在初始化选取连接时初始化，无法达到负载均衡的效果。

@wu-sheng wu-sheng reopened this Dec 4, 2015
@wu-sheng
Copy link
Member Author

wu-sheng commented Dec 4, 2015

应使用更合理的负载机制，随机切换可用连接，不应该选取后不替换，造成部分服务器压力过大。

@wu-sheng wu-sheng changed the title DataSenderFactory中的DataSenderChecker对于按百分比选择连接，处理不正确 DataSenderFactory中的DataSenderChecker对于按百分比选择连接，使用更合理的分散算法与策略 Dec 4, 2015
@wu-sheng
Copy link
Member Author

wu-sheng commented Dec 7, 2015

新的算法使用独立线程进行连接检测
1.释放不可用连接
2.为连接池新补充连接（连接池配额不足时）
3.连接池正常时，定期替换，保证服务器连接的负载均衡

@wu-sheng wu-sheng closed this as completed Dec 7, 2015
wu-sheng pushed a commit that referenced this issue Feb 26, 2017
@ascrutae
Merge pull request #2 from wu-sheng/feature/3.0
04a232d 
merge Feature/3.0
wu-sheng pushed a commit that referenced this issue Jan 9, 2018
@peng-yongsheng
Merge pull request #2 from YunaiV/master-5.0
441090e 
 Fix DataTTLKeeperTimer's startTimestamp and endTimestamp
wu-sheng pushed a commit that referenced this issue Feb 28, 2018
@hanahmily
Merge pull request #2 from apache/master
a393abc 
Update fork
@hepyu hepyu mentioned this issue Sep 6, 2018
Closed
@zhijunzhong zhijunzhong mentioned this issue Apr 11, 2019
Closed
@jadezhong jadezhong mentioned this issue Oct 23, 2019
Closed
4 tasks
GuoDuanLZ added a commit to GuoDuanLZ/skywalking that referenced this issue Jan 29, 2020
@GuoDuanLZ
Merge pull request apache#2 from GuoDuanLZ/grpc
93cdc2a 
Update test case
@linliaoy linliaoy mentioned this issue Sep 3, 2020
Closed
4 tasks
wu-sheng pushed a commit that referenced this issue Jun 2, 2021
@wingwong-knh
Refine backend (#2) (#7056)
da75a7f
liqiangz pushed a commit to liqiangz/skywalking that referenced this issue Jun 9, 2021
@wingwong-knh @liqiangz
Refine backend (apache#2) (apache#7056)
54ef8b7
@yuzuofu yuzuofu mentioned this issue Feb 21, 2023
Closed
3 tasks
@lijie123bes lijie123bes mentioned this issue Jul 11, 2024
Closed
3 tasks
lu-xiaoshuang pushed a commit to lu-xiaoshuang/skywalking that referenced this issue Aug 12, 2024
@wu-sheng
Rename common modules. (apache#2)
6950bb1
@lijie123bes lijie123bes mentioned this issue Oct 22, 2024
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working and you are sure it's a bug!
Projects
None yet
Milestone
1.0-alpha1
Development

No branches or pull requests
2 participants
@wu-sheng @ascrutae