Github user StephanEwen commented on the pull request:
https://github.com/apache/incubator-flink/pull/224#issuecomment-64086044
This change has probably a lot of implications. We may loose some
normalization of the paths by skipping that call.
Also, in commit 8b39ba9e45145a30c2b95c47d7b03bc3ed2a8a3a I played with
various versions of paths. Turns out that it is very easy to forget a slash in
the scheme part of the URL, which is currently caught by the check for absolute
paths. We may loose that as well.
The `get()` method only obtains the file system, it does not actually open
a file stream. The only important parts here are the scheme and the authority
parts of the URI. Is it any problem when interpreting the path as absolute?
---
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.
---
