-
Notifications
You must be signed in to change notification settings - Fork 59
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
Process ordering #174
Process ordering #174
Conversation
@iffyuva can you have a look? |
@gnufied i looked at this PR today morning, and PR looks good. only concern is backward compatibility. let me try out this branch for couple of projects, and i will let you know. thanks. |
I think it should be 100% backward compatible, the only change that users not using this feature will see is - 1s delay between starting their processes. I added that because, sometimes when processes are dependent - it is a good idea to introduce some delay between starting them. But we can probably drop it too. |
Just curious. Can two processes have the same index?
|
Yeah they can, in which case they would be ordered by order in which they were parsed. In fact if a process has to no index, currently it is assigned 0 index. |
It may be worth assigning higher index for processes without explicit index, rather than 0. @nisanth074 what you think? |
@gnufied If I add a process which doesn't depend on anything else
I'd intuitively expect the process to be started immediately. Hence I feel defaulting to 0 index for processes without explicit index is fine. |
couple of things to discuss:
|
Implement support for adding index to processes such that they can be started in specific order.