hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bibin A Chundatt (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (YARN-4967) AbstractService should set state only after state change
Date Sun, 17 Apr 2016 09:37:25 GMT

     [ https://issues.apache.org/jira/browse/YARN-4967?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Bibin A Chundatt updated YARN-4967:
-----------------------------------
    Description: 
As per the discussion in YARN-3971 the we should be setting the service state to STARTED only
after serviceStart() 
Currently {{AbstractService#start()}} is set
{noformat} 
     if (stateModel.enterState(STATE.STARTED) != STATE.STARTED) {
        try {
          startTime = System.currentTimeMillis();
          serviceStart();
..
 }
{noformat}
enterState sets the service state to proposed state. So in {{service.getServiceState}} in
{{serviceStart()}} will return STARTED .

  was:
As per the discussion in YARN-3971 the we should be setting the service state to STARTED only
after serviceStart() same for INiT too. Will be able to handle cases based on service state.
Thoughts?


> AbstractService should set state only after state change
> --------------------------------------------------------
>
>                 Key: YARN-4967
>                 URL: https://issues.apache.org/jira/browse/YARN-4967
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Bibin A Chundatt
>
> As per the discussion in YARN-3971 the we should be setting the service state to STARTED
only after serviceStart() 
> Currently {{AbstractService#start()}} is set
> {noformat} 
>      if (stateModel.enterState(STATE.STARTED) != STATE.STARTED) {
>         try {
>           startTime = System.currentTimeMillis();
>           serviceStart();
> ..
>  }
> {noformat}
> enterState sets the service state to proposed state. So in {{service.getServiceState}}
in {{serviceStart()}} will return STARTED .



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

Mime
View raw message