[GitHub] spark pull request: [SPARK-11859][Mesos] SparkContext accepts inva...

2015-11-25 Thread toddwan
Github user toddwan commented on the pull request: https://github.com/apache/spark/pull/9886#issuecomment-159836411 @andrewor14 I am not sure about the cause of the unit test failure, but the failed test reported in the latest test build can pass on my machine. --- If your project

[GitHub] spark pull request: [SPARK-11859][Mesos] SparkContext accepts inva...

2015-11-24 Thread toddwan
Github user toddwan commented on the pull request: https://github.com/apache/spark/pull/9886#issuecomment-159497221 retest this please --- 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

[GitHub] spark pull request: [SPARK-11859][Mesos] SparkContext accepts inva...

2015-11-24 Thread toddwan
Github user toddwan commented on the pull request: https://github.com/apache/spark/pull/9886#issuecomment-159313797 A new commit has been appended to the PR. Now a warning message is printed first if the given mesos Master URL starts with `zk://`, then a correct mesos Master URL is

[GitHub] spark pull request: [SPARK-11880][Windows][Spark Submit] bin/load-...

2015-11-21 Thread toddwan
Github user toddwan commented on the pull request: https://github.com/apache/spark/pull/9863#issuecomment-158661692 I guess users on Windows platform seldom touch `spark-env.cmd`, and have lots of workarounds if they ran into this issue. --- If your project is set up for it, you can

[GitHub] spark pull request: [SPARK-11859][Mesos] SparkContext accepts inva...

2015-11-21 Thread toddwan
GitHub user toddwan opened a pull request: https://github.com/apache/spark/pull/9886 [SPARK-11859][Mesos] SparkContext accepts invalid Master URLs in the form zk://host:port for a multi-master Mesos cluster using ZooKeeper * According to below doc and validation logic in

[GitHub] spark pull request: [SPARK-11880][Windows][Spark Submit] bin/load-...

2015-11-20 Thread toddwan
GitHub user toddwan opened a pull request: https://github.com/apache/spark/pull/9863 [SPARK-11880][Windows][Spark Submit] bin/load-spark-env.cmd loads spark-env.cmd from wrong directory * On windows the `bin/load-spark-env.cmd` tries to load `spark-env.cmd` from `%~dp0..\..\conf