Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10536660
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -130,6 +130,18 @@ class SparkContext(
val isLocal = (master == "local" ||
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37453908
Merged build finished.
---
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/119#issuecomment-37453910
One or more automated tests failed
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13134/
---
If your p
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10535551
--- Diff: core/src/test/scala/org/apache/spark/TestUtils.scala ---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37450691
About turning this on by default, I'm afraid it will mess up uses of Spark
inside a servlet container or similar. Maybe we can keep it off at first.
---
If your project is
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37446754
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 hav
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37446755
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 t
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10532081
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -130,6 +130,18 @@ class SparkContext(
val isLocal = (master == "local" || m
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10532017
--- Diff: docs/configuration.md ---
@@ -393,6 +393,16 @@ Apart from these, the following properties are also
available, and may be useful
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10531950
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -767,6 +781,20 @@ class SparkContext(
case _ =>
path
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10531841
--- Diff: core/src/test/scala/org/apache/spark/TestUtils.scala ---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10531878
--- Diff: core/src/test/scala/org/apache/spark/TestUtils.scala ---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or
Github user mateiz commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10531729
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -767,6 +781,20 @@ class SparkContext(
case _ =>
path
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37439013
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 t
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37439158
Merged build finished.
---
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/119#issuecomment-37439159
One or more automated tests failed
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13133/
---
If your p
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37439012
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 hav
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37437138
@mateiz do you mind taking a look at this? Also, how would you feel about
turning this on by default? I think in pretty much every case we'd want the
jars added to be vis
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10528875
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -130,6 +130,16 @@ class SparkContext(
val isLocal = (master == "local" || ma
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37434570
Hey @sryza thanks for the review. I responded to your comment and also
added a unit and a doc change test to clarify the behavior wrt threads.
---
If your project is set
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10527985
--- Diff: docs/configuration.md ---
@@ -393,6 +394,16 @@ Apart from these, the following properties are also
available, and may be useful
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10527981
--- Diff: docs/configuration.md ---
@@ -430,7 +441,7 @@ Apart from these, the following properties are also
available, and may be useful
spark.broadcas
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10507796
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -130,6 +130,16 @@ class SparkContext(
val isLocal = (master == "local" ||
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10507644
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -130,6 +130,16 @@ class SparkContext(
val isLocal = (master == "local" || ma
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10506498
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -130,6 +130,16 @@ class SparkContext(
val isLocal = (master == "local" ||
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10506504
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -130,6 +130,16 @@ class SparkContext(
val isLocal = (master == "local" ||
Github user pwendell commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10506499
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -130,6 +130,16 @@ class SparkContext(
val isLocal = (master == "local" ||
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10506355
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -130,6 +130,16 @@ class SparkContext(
val isLocal = (master == "local" || ma
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10506306
--- Diff: docs/configuration.md ---
@@ -393,6 +394,16 @@ Apart from these, the following properties are also
available, and may be useful
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10506244
--- Diff: docs/configuration.md ---
@@ -375,7 +376,7 @@ Apart from these, the following properties are also
available, and may be useful
spark.akka.heartb
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10506248
--- Diff: docs/configuration.md ---
@@ -430,7 +441,7 @@ Apart from these, the following properties are also
available, and may be useful
spark.broadcast.b
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10506241
--- Diff: docs/configuration.md ---
@@ -111,6 +111,7 @@ Apart from these, the following properties are also
available, and may be useful
it if you confi
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/119#discussion_r10506221
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -726,6 +736,8 @@ class SparkContext(
* Adds a JAR dependency for all tasks to be
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37377166
One or more automated tests failed
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13124/
---
If your p
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37377165
Merged build finished.
---
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/119#issuecomment-37377100
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 hav
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37377101
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 t
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37377049
@velvia @sryza... I added some tests for this. Do you remember what the
corner cases were?
---
If your project is set up for it, you can reply to this email and have you
Github user velvia commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37269362
Hi @pwendell, thanks for moving this over. I think there are some minor
changes that may be necessary to complete it, some corner cases, but I'll have
to dig up to find w
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37267178
One or more automated tests failed
Refer to this link for build results:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/13108/
---
If your p
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37267177
Merged build finished.
---
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/119#issuecomment-37267134
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 t
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/119#issuecomment-37267133
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 hav
GitHub user pwendell opened a pull request:
https://github.com/apache/spark/pull/119
SPARK-1230: [WIP] Enable SparkContext.addJars() to load classes not in
CLASSPATH
Reopens and up-merges @velvia's patch previously against the old repo:
https://github.com/apache/incubator-spark/
44 matches
Mail list logo
