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

State model for task manager - independent of RTS -- design discussion #243

Closed
vivek-bala opened this issue Jun 26, 2018 · 5 comments
Closed

Comments

@vivek-bala
Copy link
Contributor

To keep the interaction between EnTK and an RTS independent of the specific RTS used

@mturilli
Copy link
Contributor

If we get github discussions, this should be transitioned there.

@iparask
Copy link
Contributor

iparask commented Dec 7, 2020

Does this have to do with the task states or the RTS state? I've realized that EnTK does not distinguish between the final states of an RTS. Would it make sense to map the state of the RTS to internal states?

@mturilli
Copy link
Contributor

mturilli commented Dec 7, 2020

This refers to separating by design (and eventually implementation) EnTK and RTS. Originally, this was derived from the requirements to use other task execution systems, e.g., Dask and others that now have disappeared. Today, this requirement is less urgent and I would suggest to de-prioritize against the one of designing EnTK for better performance. Comments, suggestions?

@andre-merzky
Copy link
Member

I agree with Matteo. I mean, we probably want to make sure that adding another RTS is still possible, but it is not highest priority right now.

I've realized that EnTK does not distinguish between the final states of an RTS. Would it make sense to map the state of the RTS to internal states?

IMHO distinguishing final states is a necessity. If that is done via mapping into an RE state model or by having the RE state model expand the RTS/RP state model is really an implementation detail. Mapping will be required once we add a second RTS, until then it does not matter...

@iparask
Copy link
Contributor

iparask commented Jan 12, 2021

moved in discussions

@iparask iparask closed this as completed Jan 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants