Github user guoxu1231 commented on the pull request:

    https://github.com/apache/spark/pull/10464#issuecomment-167760455
  
    @srowen, thanks for the review. I combined the the resource release logic 
from finally block and onStop(), and use synchronized block to avoid possible 
release conflicts with restart thread(almost impossible, but in case some very 
rare circumstance). 
    Receiver usually keep running very long time, we must be very careful about 
the resource creation and release.  Thanks for the patient review scrowen :)


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