Github user mxm closed the pull request at:
https://github.com/apache/flink/pull/1235
---
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
Github user mxm commented on the pull request:
https://github.com/apache/flink/pull/1235#issuecomment-163220976
Closing this because of the raised objections.
---
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 projec
Github user mxm commented on the pull request:
https://github.com/apache/flink/pull/1235#issuecomment-146330660
@uce This was not about enforcing HDFS, rather disallowing file.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user mxm commented on the pull request:
https://github.com/apache/flink/pull/1235#issuecomment-146330037
The Hadoop configuration may also be set through resource files in the
classpath. IMHO throwing an exception is only acceptable when we're sure no
configuration has been loa
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
Github user uce commented on the pull request:
https://github.com/apache/flink/pull/1235#issuecomment-146297878
PS Enforcing hdfs:// would also not allow local testing. I saw the warning
log today when I was testing Till's HA PR.
---
If your project is set up for it, you can reply to
Github user mxm commented on the pull request:
https://github.com/apache/flink/pull/1235#issuecomment-146204887
Outside of the Hadoop world. Yes.
Let's change this to check whether any kind of Hadoop configuration has
been loaded previously.
---
If your project is set up for
Github user uce commented on the pull request:
https://github.com/apache/flink/pull/1235#issuecomment-146201155
"file://" can also be a distributed file system in certain setups.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
GitHub user mxm opened a pull request:
https://github.com/apache/flink/pull/1235
[FLINK-2823][yarn] throw exception on file as default file system
Without a distributed file system, Flink cannot distribute its files
correctly. Thus, we can throw an exception in case we discover file
