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

sequential job execution #60

Closed
tbrand opened this issue Apr 13, 2018 · 3 comments
Closed

sequential job execution #60

tbrand opened this issue Apr 13, 2018 · 3 comments

Comments

@tbrand
Copy link
Owner

tbrand commented Apr 13, 2018

Currently all jobs at depends_on are executed concurrently.
But sometimes we want to do it sequentially.
(The idea is needed at least here)

You may know you can do it like

some_job:
  depends_on:
    - sub_job1

sub_job1:
  depends_on:
    - sub_job2

But when we remove the sub_job1 from it, the pipeline will be broken.
So it's not good at the view point of flexibility and stability.

How about adding this function to neph.yaml?

some_job:
  depends_on:
    - sub_job1
    - sub_job2
  seq: true

When the seq config is true, the series of jobs on depends_on will be executed sequentially.
The default is false.

What do you guys think?

@notramo
Copy link
Contributor

notramo commented Apr 13, 2018

I had this idea also, and I agree.
I am now working on the new config format (which is not pushed yet). It will be included in it. Unfortunately, the progress is slow, because it requires very careful designing and coding if we want to get very good quality software at the end (I am rewriting it from scratch, because it was written with wrong programming practices). I don't want to spend time on the current version, because it would delay the new branch more.
If you can't wait for the new version, you can implement this feature in the current.

@tbrand
Copy link
Owner Author

tbrand commented Apr 13, 2018

Awesome.
Yup, I can wait for it. Thanks.

@tbrand tbrand closed this as completed Apr 13, 2018
@notramo
Copy link
Contributor

notramo commented Jun 11, 2018

Added to the redesigned version: 15d9c86

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants