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

Option for disabling parallel execution #13

Closed
vaclav-easymarkit opened this issue Jan 20, 2016 · 2 comments
Closed

Option for disabling parallel execution #13

vaclav-easymarkit opened this issue Jan 20, 2016 · 2 comments

Comments

@vaclav-easymarkit
Copy link

It could be helpful especially for the machines with weak CPU performance. I could try to prepare pull-request if you like to.

@zdenekdrahos
Copy link
Member

I wouldn't specify number of tasks. I see two main use cases:

  • run everything in parallel (current behaviour)
  • don't use parallelism at all

So I think that simple switch would be good enough. Just like in --output:

# run parallel tasks
phpqa
phpqa --execution parallel

# don't use parallelism
phpqa --execution single
phpqa --execution something_else_than_word_parallel

If --execution equals parallel then ParallelExec is used. Otherwise tasks are executed sequentially with ExecStack.

No BC is introduced (parallel execution is default option) and the solution is extensible in the future (if --execution is integer then it can be used as number of parallel processes). It's simple change compared to dividing processes into parallel groups.

@vaclav-easymarkit
Copy link
Author

Yes, you are right, it will be sufficient.

@zdenekdrahos zdenekdrahos changed the title Possibility to define count of parallel taks Option for disabling parallel execution Jan 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants