Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/1235#issuecomment-146303645
I don't think that we can merge this change. We can not assume that
file:/// is an invalid path.
Also, I don't think there is a good way to check whether the hadoop
configuration has been loaded or whether the user is just running on the
default configuration.
I think the only thing we can do is throw an exception if none of the
common environment variable are set. But even then, users can put the path to
the hadoop configuration into the flink-conf.yaml.
But I think its okay, a workaround for these users would be to just set one
of the environment variables to make the exception go away.
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
