Use amphp/process for parallel execution.

jigarius / drall

A tool to run drush commands on multi-site Drupal installations.

GNU General Public License v3.0

33 stars 5 forks source link

Use amphp/process for parallel execution. #51

Closed jigarius closed 2 years ago

jigarius commented 2 years ago

Tasks

[x] Use amphp/process libraries
- [x] Limit max-workers to match --drall-workers value.
- [x] Update tests
[x] Remove all code from the old implementation of parallel execution
- [x] Remove Runner/*
- [x] Remove Queue/*
[x] Update README, if required.

jigarius commented 2 years ago

I had to remove some of the unit tests in favor of integration tests. With this 3rd party library in place, I think it's easier and time-saving to use integration tests.

Next, I will:

Ensure proper test coverage.
Remove all stale code related to the home-grown parallel execution.

jigarius commented 2 years ago

Finally! I think this PR is ready to get merged 🕺🏽