Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/3561#issuecomment-66383770
Even if we're only going to support the `file://` scheme for now, we should
add a check that the URI actually uses the `file://` scheme before attempting
to strip it off and treat the path as a local path. If we don't do this, then
we risk bugs where someone passes a `hdfs://` path but we read the file from a
local path.
(I think that this was implicit in my comments (and Andrew's), but I just
wanted to say it a bit more explicitly to avoid any confusion).
---
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]
