Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8649
---
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 enab
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138983239
LGTM, merging.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138855538
Merged build finished. Test PASSed.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138855539
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138855250
[Test build #42194 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42194/console)
for PR 8649 at commit
[`08f7c9a`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138808603
[Test build #42194 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42194/consoleFull)
for PR 8649 at commit
[`08f7c9a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138808192
Merged build triggered.
---
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138808209
Merged build started.
---
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
Github user zjffdu commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138808084
Thanks @vanzin
update the patch with re-word message.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138739307
@zjffdu could you re-word the message? I'd suggest something like:
Could not find jar containing Spark classes. The jar can be defined
using the
spark
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138738866
Merged build finished. Test PASSed.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138738867
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138738797
[Test build #42155 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42155/console)
for PR 8649 at commit
[`39fe23a`](https://github.
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138716140
@vanzin makes sense if its only going to impact spark developers (thought
it might also be a user facing exception).
---
If your project is set up for it, you can reply
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138715312
[Test build #42155 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42155/consoleFull)
for PR 8649 at commit
[`39fe23a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138714778
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138714751
Merged build triggered.
---
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 h
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138714001
ok to test
---
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
enab
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138708571
> Wouldn't it make more sense to check for this in SparkSubmitArguments or
maybe the yarn specific validation code?
I'm not sure there's a common enough place whe
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8649#discussion_r38982863
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -1045,7 +1045,8 @@ object Client extends Logging {
s"in favor of t
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138706578
Wouldn't it make more sense to check for this in SparkSubmitArguments or
maybe the yarn specific validation code?
---
If your project is set up for it, you can reply to
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/8649#discussion_r38887638
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -1045,7 +1045,8 @@ object Client extends Logging {
s"in favor o
Github user zjffdu commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138411720
Paste the new exception after this patch.
```
15/09/08 10:05:52 ERROR SparkContext: Error initializing SparkContext.
org.apache.spark.SparkException: Could not
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8649#issuecomment-138410759
Can one of the admins verify this patch?
---
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 pr
GitHub user zjffdu opened a pull request:
https://github.com/apache/spark/pull/8649
SPARK-10481 SPARK_PREPEND_CLASSES make spark-yarn related jar could nâ¦
Throw a more readable exception. Please help review. Thanks
You can merge this pull request into a Git repository by run
25 matches
Mail list logo
