Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3740#issuecomment-67608353
[Test build #24635 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24635/consoleFull)
for PR 3740 at commit
[`734bac9`](https://githu
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/3740#discussion_r22093377
--- Diff: core/src/main/scala/org/apache/spark/rdd/RDD.scala ---
@@ -1174,6 +1174,14 @@ abstract class RDD[T: ClassTag](
* Save this RDD as a text file
Github user tnachen commented on the pull request:
https://github.com/apache/spark/pull/60#issuecomment-67607923
Seems like the original author is no longer going to work on this, I'll
take this up again.
---
If your project is set up for it, you can reply to this email and have your
Github user tdna commented on the pull request:
https://github.com/apache/spark/pull/60#issuecomment-67607842
If somebody uses mesos with more framework it is a very important feature
(like us), please somebody file an issue on the Jira.
---
If your project is set up for it, you can
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2956#issuecomment-67607694
[Test build #24629 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24629/consoleFull)
for PR 2956 at commit
[`be7c1fa`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2956#issuecomment-67607704
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3734#issuecomment-67607387
[Test build #24634 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24634/consoleFull)
for PR 3734 at commit
[`cc74727`](https://githu
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/3610#issuecomment-67607352
LGTM except minor inline comments.
---
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
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/3734#discussion_r22092977
--- Diff:
network/shuffle/src/main/java/org/apache/spark/network/shuffle/ExternalShuffleBlockManager.java
---
@@ -60,8 +61,11 @@
private final Transpo
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092935
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092959
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092957
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092963
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/stat/impl/MultivariateGaussian.scala
---
@@ -0,0 +1,39 @@
+/*
+ * Licensed to the Apache Softwar
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092960
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092939
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092937
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092942
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092953
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092956
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092938
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092962
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/stat/impl/MultivariateGaussian.scala
---
@@ -0,0 +1,39 @@
+/*
+ * Licensed to the Apache Softwar
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092955
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092934
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092952
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092954
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092927
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3738#issuecomment-67607055
[Test build #24633 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24633/consoleFull)
for PR 3738 at commit
[`82e02cc`](https://githu
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092924
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092941
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092918
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModel.scala
---
@@ -0,0 +1,94 @@
+/*
+ * Licensed to the Apache Softwa
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092908
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/DenseGmmEM.scala ---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092933
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092923
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModel.scala
---
@@ -0,0 +1,94 @@
+/*
+ * Licensed to the Apache Softwa
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092920
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModel.scala
---
@@ -0,0 +1,94 @@
+/*
+ * Licensed to the Apache Softwa
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092926
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModelEM.scala
---
@@ -0,0 +1,248 @@
+/*
+ * Licensed to the Apache Sof
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092915
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModel.scala
---
@@ -0,0 +1,50 @@
+/*
+ * Licensed to the Apache Softwa
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092921
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModel.scala
---
@@ -0,0 +1,94 @@
+/*
+ * Licensed to the Apache Softwa
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092909
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/DenseGmmEM.scala ---
@@ -0,0 +1,65 @@
+/*
+ * Licensed to the Apache Software Foundat
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092917
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModel.scala
---
@@ -0,0 +1,94 @@
+/*
+ * Licensed to the Apache Softwa
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/3022#discussion_r22092919
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/clustering/GaussianMixtureModel.scala
---
@@ -0,0 +1,94 @@
+/*
+ * Licensed to the Apache Softwa
Github user aarondav commented on a diff in the pull request:
https://github.com/apache/spark/pull/3740#discussion_r22092697
--- Diff: core/src/main/scala/org/apache/spark/rdd/RDD.scala ---
@@ -1174,6 +1174,14 @@ abstract class RDD[T: ClassTag](
* Save this RDD as a text fil
Github user aarondav commented on a diff in the pull request:
https://github.com/apache/spark/pull/3734#discussion_r22092572
--- Diff:
network/shuffle/src/main/java/org/apache/spark/network/shuffle/ExternalShuffleBlockManager.java
---
@@ -60,8 +61,11 @@
private final Transp
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3607#issuecomment-67606014
[Test build #24628 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24628/consoleFull)
for PR 3607 at commit
[`2b27928`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3607#issuecomment-67606019
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/3739#issuecomment-67606021
> The trickiness of this trait ordering might be an argument against this
approach.
Yeah. But if it's well documented, I won't be against it. Really a
convenient
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3738#issuecomment-67605600
[Test build #24632 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24632/consoleFull)
for PR 3738 at commit
[`217062a`](https://githu
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3741#issuecomment-67605602
[Test build #24631 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24631/consoleFull)
for PR 3741 at commit
[`3d6631f`](https://githu
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/3739#issuecomment-67605537
> Ideally, a test should not modify environment shared by other tests. Or
we cannot make the tests run in parallel.
We can achieve parallelism by running multip
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/3739#issuecomment-67605441
That looks like a [legitimate test
failure](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24618/testReport/).
It looks like the issue there was th
GitHub user jongyoul opened a pull request:
https://github.com/apache/spark/pull/3741
[SPARK-4465] runAsSparkUser doesn't affect TaskRunner in Mesos environme...
...nt at all.
- fixed a scope of runAsSparkUser from MesosExecutorDriver.run to
MesosExecutorBackend.launchTask
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/3739#issuecomment-67605399
Ideally, a test should not modify environment shared by other tests. Or we
cannot make the tests run in parallel.
---
If your project is set up for it, you can reply to
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/3738#issuecomment-67605336
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 f
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/3429#discussion_r22092141
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveInspectors.scala ---
@@ -119,12 +269,44 @@ private[hive] trait HiveInspectors {
S
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/3429#discussion_r22092047
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveInspectors.scala ---
@@ -119,12 +269,44 @@ private[hive] trait HiveInspectors {
S
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/3737#issuecomment-67604433
The deprecation warning idea was good; it turns out that there's a pending
deprecation which will change the semantics of one of the methods we use, so I
upgraded the c
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3737#issuecomment-67604418
[Test build #24630 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24630/consoleFull)
for PR 3737 at commit
[`f02935d`](https://githu
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/3721#discussion_r22091865
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/util/WriteAheadLogManager.scala
---
@@ -125,7 +125,7 @@ private[streaming] class WriteAheadLogM
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3157#issuecomment-67604251
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3157#issuecomment-67604244
[Test build #24626 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24626/consoleFull)
for PR 3157 at commit
[`cd50b2b`](https://gith
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/3721#discussion_r22091804
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/util/WriteAheadLogManager.scala
---
@@ -125,7 +125,7 @@ private[streaming] class WriteAheadLogM
Github user scwf commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67604099
@marmbrus, is this ok to go?
---
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 th
Github user witgo commented on the pull request:
https://github.com/apache/spark/pull/3729#issuecomment-67603907
The PR is temporarily close.
---
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 witgo closed the pull request at:
https://github.com/apache/spark/pull/3729
---
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 enabl
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/3684
---
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 JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/3684#issuecomment-67603755
This looks good to me, so I'm going to merge this into `master` (1.3.0),
`branch-1.2` (1.2.1) and `branch-1.1`, (1.1.2). Thanks!
---
If your project is set up for it,
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67603749
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67603746
[Test build #24625 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24625/consoleFull)
for PR 3718 at commit
[`e0d6b5d`](https://gith
Github user JoshRosen closed the pull request at:
https://github.com/apache/spark/pull/3681
---
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 e
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2956#issuecomment-67603039
[Test build #24629 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24629/consoleFull)
for PR 2956 at commit
[`be7c1fa`](https://githu
Github user mengxr closed the pull request at:
https://github.com/apache/spark/pull/3106
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3740#issuecomment-67602317
[Test build #24624 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24624/consoleFull)
for PR 3740 at commit
[`ca03559`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3740#issuecomment-67602319
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/3677#issuecomment-67602032
Had an offline discussion with @witgo . The iterator interface looks good
to me, but creating composite methods could be avoided.
---
If your project is set up for it, yo
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/3729#issuecomment-67601970
Had an offline discussion with @witgo . The current implementation takes
advantage of Parquet's serialization over Java's. However, this is too special
for this particular
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2854#issuecomment-67601642
[Test build #24627 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24627/consoleFull)
for PR 2854 at commit
[`95ae4db`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/2854#issuecomment-67601648
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/2848#issuecomment-67601524
This looks good to me, so I'm ready to merge it. One final little thing,
though: since we have two separate PRs that address SPARK-3967, I'd like to
create a new JIRA
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3607#issuecomment-67601463
[Test build #24628 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24628/consoleFull)
for PR 3607 at commit
[`2b27928`](https://githu
Github user WangTaoTheTonic commented on a diff in the pull request:
https://github.com/apache/spark/pull/3607#discussion_r22090815
--- Diff: docs/running-on-yarn.md ---
@@ -22,6 +22,14 @@ Most of the configs are the same for Spark on YARN as
for other deployment modes
P
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/3731#discussion_r22090703
--- Diff: docs/job-scheduling.md ---
@@ -56,6 +56,112 @@ the same RDDs. For example, the
[Shark](http://shark.cs.berkeley.edu) JDBC serve
queries. In futur
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/3532
---
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 JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/3532#issuecomment-67600571
This looks good to me (most of the diff is just an indentation change), so
I'm going to merge this to `master` (1.3.0). Thanks!
Just out of curiosity, what mot
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/2854#issuecomment-67600250
[Test build #24627 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24627/consoleFull)
for PR 2854 at commit
[`95ae4db`](https://githu
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/2854#issuecomment-67600208
Jenkins, 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 ha
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/3607#discussion_r22090369
--- Diff: docs/running-on-yarn.md ---
@@ -22,6 +22,14 @@ Most of the configs are the same for Spark on YARN as
for other deployment modes
Property Na
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3607#issuecomment-67599686
[Test build #24623 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24623/consoleFull)
for PR 3607 at commit
[`b7acbb2`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/3607#issuecomment-67599691
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3157#issuecomment-67599082
[Test build #24626 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24626/consoleFull)
for PR 3157 at commit
[`cd50b2b`](https://githu
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/3157#issuecomment-67598898
test 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 feature
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/3680#issuecomment-67598867
LGTM. Merged into master. I've assigned the Python API JIRA to you and set
the target version to 1.3.0. So it is not in a hurry. Happy holidays!
---
If your project is se
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/3680
---
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 liyezhang556520 commented on the pull request:
https://github.com/apache/spark/pull/2134#issuecomment-67598353
Hi @andrewor14 ,
[PR#3629](https://github.com/apache/spark/pull/3629) solved the problem
that I pointed out in your original patch
[PR#1165](https://github.co
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3718#issuecomment-67598149
[Test build #24625 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24625/consoleFull)
for PR 3718 at commit
[`e0d6b5d`](https://githu
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/3712
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/3712#issuecomment-67597165
Thanks! Merged to master.
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/3429
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/3740#issuecomment-67597016
[Test build #24624 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24624/consoleFull)
for PR 3740 at commit
[`ca03559`](https://githu
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/3429#issuecomment-67596996
Thanks! Merged to master.
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/3348
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/3348#issuecomment-67596915
Thanks! Merged to master.
---
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
1 - 100 of 490 matches
Mail list logo