Skip to content
This repository has been archived by the owner on Jan 21, 2021. It is now read-only.

External pool #3

Merged
merged 3 commits into from
Feb 7, 2015
Merged

Conversation

jinty
Copy link

@jinty jinty commented Oct 3, 2013

Allow the pool implementation to be overridden and provide status. Minimum required to support:
https://github.com/vanguardistas/van.resumelb

jinty added 3 commits October 4, 2013 00:27
git-svn-id: svn+ssh://svn.zope.org/repos/main/zc.resumelb/branches/jinty-external-pool@127383 62d5b8a3-27da-0310-9561-8e5933582275
git-svn-id: svn+ssh://svn.zope.org/repos/main/zc.resumelb/branches/jinty-external-pool@127384 62d5b8a3-27da-0310-9561-8e5933582275
…rbitrary

data over the status server.


git-svn-id: svn+ssh://svn.zope.org/repos/main/zc.resumelb/branches/jinty-external-pool@127385 62d5b8a3-27da-0310-9561-8e5933582275
@jimfulton jimfulton merged commit 3698eb9 into zopefoundation:master Feb 7, 2015
jimfulton pushed a commit that referenced this pull request Feb 7, 2015
@jimfulton
Copy link
Member

Much thanks. This was a great PR. Sorry I took so long to merge it.

@jinty
Copy link
Author

jinty commented Feb 11, 2015

no worries, thanks for zc.resumelb!

I prefer slow acceptance of pull requests than bad software any day of the week.

@jimfulton
Copy link
Member

:)

BTW, I'm working on a resume-less pool that just selects workers with minimum backlogs.

If you have anything I should consider instead... :)

It turns out that we have lots of smaller applications where resumes get in the way, but having a simple ZooKeeper-aware back-end LB is very useful.

@jinty
Copy link
Author

jinty commented Feb 16, 2015

On Wed, Feb 11, 2015 at 01:43:10PM -0800, Jim Fulton wrote:

:)

BTW, I'm working on a resume-less pool that just selects workers with minimum backlogs.

If you have anything I should consider instead... :)

It turns out that we have lots of smaller applications where resumes get in the way, but having a simple ZooKeeper-aware back-end LB is very useful.

We don't really need the resumes, but I've never found that it "got in
the way".

I am still using my own pool implementation:

https://github.com/vanguardistas/van.resumelb

Not sure if that fits into "something else you should consider" but
it has been extremely useful and stable. It makes upgrades very smooth
as we can switch users from one version to another more or less
instantaneously.

In future, I may need to work in this area again to start sending
different requests to more specialized backend servers. So I will end up
classifying on:

(service, version, classifier)

I.e. running multiple versions of multiple services behind a singe
resumelb. I suppose you could run multiple resumelbs, one per service,
but that may make the system harder to run/monitor.

Brian Sutherland

@jimfulton
Copy link
Member

Ah right. Well, I've added a simple class-less pool that does least-connection load balancing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants