Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/607#issuecomment-42030224
I disagree. I would rather see this fixed properly. If we leave it with
your patch and I disable this option then I still leak filesystems, it seems
bad to leave a leak in the code that we know about. If you want to use your
patch and clean up properly that would be ok.
I would personally still like to see the doAs moved up. Some of the
reasoning you mentioned is why the doAs should go up into the
CoarseGrainExecutorBackend (and other Backends) like I mentioned above.
I'll make a patch for this that moves the doAs and post it hopefully later
this morning.
---
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.
---
