Github user ilganeli commented on the pull request:

    https://github.com/apache/spark/pull/5277#issuecomment-87946825
  
    Ok so it looks like we have a simple fix for the time being, in the 
long-term, so we aren't assigning to the ```stopped``` before the entire 
process is complete, I'd prefer a slightly different solution. I would add a 
second AtomicBoolean ```stopping``` and make the if check to be ```if(!stopping 
&& !stopped)```. Stopping would be set true in the beginning of the process and 
stopped would be set true at the end. This would guard against incomplete 
stops. 


---
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