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

Make ConnectionSet processing round-robin #38

Open
eile opened this issue Sep 20, 2013 · 1 comment
Open

Make ConnectionSet processing round-robin #38

eile opened this issue Sep 20, 2013 · 1 comment
Assignees
@delyas
Labels
optimization
Milestone
1.2

Comments

@eile
Copy link
Member

eile commented Sep 20, 2013

Reported by @delyas: The current ConnectionSet::select processing favors the first connections, which causes timeouts when a set of clients pushes a lot of requests fast (mapping) onto a single node, which will then first serve the 'first' clients.

Processing this in a round-robin fashion will ensure that all clients make progress and should speed up the rcv thread since commands are fully read already.
@ghost ghost assigned delyas Sep 20, 2013
@eile
Copy link
Member Author

eile commented Sep 20, 2013

Attention needs to be paid to the >64 connection threads on Windows.

eile pushed a commit to eile/Collage that referenced this issue Oct 17, 2013
@tribal-tec
Merge pull request Eyescale#38 from rdumusc/master
ae928bf 
Add FindPoppler.cmake
eile pushed a commit to eile/Collage that referenced this issue Jan 22, 2014
Implement Unix side of Eyescale#38: make connectionset round-robin
e4c8d2b
eile pushed a commit to eile/Collage that referenced this issue Jan 22, 2014
Fix OS X build, make Eyescale#38 test more robust
f347b3f
tribal-tec added a commit that referenced this issue Jan 23, 2014
@tribal-tec
Merge pull request #76 from eile/master
8cd90e5 
Implement Unix side of #38: make connectionset round-robiny
@eile eile mentioned this issue Mar 12, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@delyas delyas

Labels
optimization
Projects
None yet
Milestone
1.2
Development

No branches or pull requests
2 participants
@eile @delyas