Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/4871#issuecomment-77040982
Ah, @srowen @o-mdr I misunderstood what you guys were saying. I think what
@o-mdr meant was that moving the two lines after `stopped = true` wouldn't
change anything because we in a synchronized block anyway. Yes, that is true,
but I also agree with @srowen that it's a slightly more readable reorganization
with no change in behavior. This is a super minor point, but I made the change
anyway.
---
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]
