The Wayback Machine - https://web.archive.org/web/20200914155315/https://github.com/google/crmint/pull/14
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

Improve job scheduling #14

Open
wants to merge 3 commits into
base: dev
from
Open

Conversation

@dulacp
Copy link
Member

dulacp commented Jul 19, 2018

The new implementation is still decoupling the Worker and Job business logics. The difference comes from the flexibility given to the views.Task controller who can now schedule jobs as soon as the _enqueue(...) method is called.

NB: Tests need to consume the generator otherwise it will be garbage collected and a GeneratorExit exception will be raised. It's Python smart but obscure optimizations :)

Cheers

TODO:

  • Fix the stopping state in case of parallelization
  • Fix the success state in case of parallelization
@dulacp dulacp requested a review from Slony Jul 19, 2018
dulacp added 2 commits Jul 19, 2018
The new implementation is still decoupling the Worker and Job business
logics. The difference comes from the flexibility given to the
`views.Task` controller who can now schedule jobs as soon as the
`_enqueue(...)` method is called.
Tests need to consume the generator otherwise it will be garbage
collected and a GeneratorExit exception will be raised. It's Python
smart but obscure optimizations :)
@dulacp dulacp force-pushed the dulacp:improve-job-scheduling branch from 2ad637b to 12f84a8 Jul 23, 2018
@dulacp
Copy link
Member Author

dulacp commented Jul 23, 2018

Please do not merge this yet, the state machine is not stable with parallelization.
I will investigate why tomorrow :)

@dulacp dulacp force-pushed the dulacp:improve-job-scheduling branch from e696b91 to 88cc5e2 Jul 24, 2018
@dulacp dulacp force-pushed the dulacp:improve-job-scheduling branch from 88cc5e2 to 35e4f57 Jul 24, 2018
@dulacp
Copy link
Member Author

dulacp commented Sep 4, 2018

Waiting for #19 to be submitted, and then rebase this PR on the dev branch.
We need the new state machine to make this yield enqueuing work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
1 participant
You can’t perform that action at this time.