[jira] [Commented] (REEF-826) Add state transition check in ResourceManagerStatus.setState

["Sergey Dudoladov (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/REEF-826?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14953037#comment-14953037 ] 

Sergey Dudoladov commented on REEF-826:
---------------------------------------

Here is the draft of the state transition diagram:

|From / To | Init  |Running |Suspend  |Done |Failed |Killed
|Init            |   ?     |    T        |     T         |    T    |     T    |    T
|Running   |          |        ?     |         T    |    T    |     T     |   T
|Suspend  |          |        T     |         ?    |           |     T     |  T
|Done        |          |              |                |     ?   |           |       |
|Failed       |        |              |               |           |      ?    |        |
|Killed        |        |              |                |          |            |    ?

"T" stands for True is the state transition is legal; empty space represents illegal transitions. 

 For the entries on the main diagonal,  if the message with the same state update may be received twice in a row, then the duplicate probably can be simply discarded.

> Add state transition check in ResourceManagerStatus.setState
> ------------------------------------------------------------
>
>                 Key: REEF-826
>                 URL: https://issues.apache.org/jira/browse/REEF-826
>             Project: REEF
>          Issue Type: Improvement
>          Components: REEF-Common
>            Reporter: Mariia Mykhailova
>            Priority: Minor
>
> Currently we don't have a check whether the state transition in {{ResourceManagerStatus.setStatus}} is legal and expected. We should add a check similar to what we have in {{EvaluatorManager}} and {{TaskStatus.setState}}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)