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.

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

Rewrite ipdevpoll's multiprocess mode to use generic worker model #1174

Closed
jmbredal opened this issue Jan 17, 2017 · 2 comments
Closed

Rewrite ipdevpoll's multiprocess mode to use generic worker model #1174

jmbredal opened this issue Jan 17, 2017 · 2 comments
Assignees
Labels
enhancement nav-ref Fronted by the NAV user reference committee optimize scale
Milestone

Comments

@jmbredal
Copy link
Collaborator

jmbredal commented Jan 17, 2017

Inspired by Apache's worker model. The multiprocess mode should spawn a number of slave processes that are assigned collector jobs by a master scheduler. The model should even allow for remote worker processes, for distributed collection.

How to launch remote worker processes is out of scope for this task, but should be considered when designing the communication between the scheduler process and the workers.


Imported from Launchpad using lp2gh.

@jmbredal jmbredal added nav-ref Fronted by the NAV user reference committee New optimize scale labels Jan 17, 2017
@sigmunau sigmunau self-assigned this Jan 19, 2017
@lunkwill42
Copy link
Member

I would like one more feature out of this: I want workers to kill themselves (only to be restarted) after having run a configurable number of jobs.

Python isn't particulary keen on releasing memory to the OS, once allocated, causing some of the workers (especially one's processing topo jobs) to increase memory usage heavily over time.

This is analogous to Apache's MaxRequestsPerChild, and will offer simple protection against inadvertent "memory leaks".

@lunkwill42 lunkwill42 added this to the 4.7.0 milestone Feb 24, 2017
@lunkwill42
Copy link
Member

Closing this, as both implementing PRs have been merged :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement nav-ref Fronted by the NAV user reference committee optimize scale
Projects
None yet
Development

No branches or pull requests

3 participants