Github user andrewor14 commented on the pull request:

    https://github.com/apache/spark/pull/8714#issuecomment-157890701
  
    @jerryshao this change looks OK, but on a high level were there any 
specific symptoms caused by this issue, other than the fact that the UI reports 
`LOADING` for some executors?
    
    Actually, I would go even further to remove `LOADING` altogether. I don't 
see how it's different from `LAUNCHING`, and both of these states are used in 
exactly one place. It's also weird that after we call `ExecutorRunner#start()` 
in `Worker` we don't set the state to `RUNNING`. These issues would subsume 
SPARK-10558.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to