Merge lp:~magentoerpconnect-core-editors/magentoerpconnect/7.0-prototype-multi-worker-notify into lp:magentoerpconnect/7.0-prototype
Proposed by
Guewen Baconnier @ Camptocamp
Status: | Work in progress |
---|---|
Proposed branch: | lp:~magentoerpconnect-core-editors/magentoerpconnect/7.0-prototype-multi-worker-notify |
Merge into: | lp:magentoerpconnect/7.0-prototype |
Prerequisite: | lp:~magentoerpconnect-core-editors/magentoerpconnect/7.0-prototype-multi-worker |
Diff against target: |
60 lines (+34/-0) 2 files modified
connectors/abstract/jobs.py (+1/-0) connectors/abstract/worker.py (+33/-0) |
To merge this branch: | bzr merge lp:~magentoerpconnect-core-editors/magentoerpconnect/7.0-prototype-multi-worker-notify |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
MagentoERPConnect core editors | Pending | ||
Review via email: mp+147387@code.launchpad.net |
This proposal supersedes a proposal from 2013-02-08.
Description of the change
Quick test using Postgresql NOTIFY.
It seems to work at first insight. It could be a way to enqueue jobs in a while the cron would act as a failback for the jobs enqueued in a dead worker.
TODO if we want to use the NOTIFY:
* retrieve the payload from the NOTIFY and assign to worker + enqueue the corresponding job
* add sort of flag on the processes which are allowed to listen the NOTIFY (either the main process in standalone mode, either the cron processes in multiprocess mode).
To post a comment you must log in.
Unmerged revisions
- 72. By Guewen Baconnier @ Camptocamp
-
[ADD] test with psql NOTIFY