Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/2848#issuecomment-67601524
This looks good to me, so I'm ready to merge it. One final little thing,
though: since we have two separate PRs that address SPARK-3967, I'd like to
create a new JIRA for this fix just in case this doesn't fix the YARN issue in
the original ticket.
Therefore, I've opened https://issues.apache.org/jira/browse/SPARK-4896.
Do you mind editing this PR's title to reference that JIRA instead? Once you
do that, I'll merge this into all of the maintenance branches and resolve the
original JIRA.
Thanks for your patience during this long review and with the build
difficulties. I think that the end result here is a significant improvement
over the old `fetchFile` code, so I'm looking forward to merging this.
---
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]
