Github user nongli commented on a diff in the pull request:
https://github.com/apache/spark/pull/9276#discussion_r43473950
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ExchangeCoordinator.scala
---
@@ -0,0 +1,251 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9373#issuecomment-152441407
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9368#issuecomment-152443414
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9368#issuecomment-152443416
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9368#issuecomment-152443336
**[Test build #44658 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44658/consoleFull)**
for PR 9368 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152445431
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152445468
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
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/9348#issuecomment-152449863
ping @marmbrus @yhuai
---
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/9313#issuecomment-152451609
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152464693
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152464747
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9241#issuecomment-152436956
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9241#issuecomment-152436900
**[Test build #44656 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44656/consoleFull)**
for PR 9241 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9241#issuecomment-152436958
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9373#issuecomment-152441681
**[Test build #44665 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44665/consoleFull)**
for PR 9373 at commit
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/9241#issuecomment-152443044
Great, sounds good to me! I do think that this patch might benefit from
more comments to explain the code and I'd be happy to help with that in a
followup PR.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9241
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9373#issuecomment-152443118
**[Test build #44666 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44666/consoleFull)**
for PR 9373 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9372
---
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
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9206#discussion_r43478074
--- Diff:
network/common/src/test/java/org/apache/spark/network/util/TransportFrameDecoderSuite.java
---
@@ -0,0 +1,173 @@
+/*
+ * Licensed to the
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9371#issuecomment-152451502
[Test build #44661 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44661/console)
for PR 9371 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9371#issuecomment-152451589
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9313#issuecomment-152451603
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9371#issuecomment-152451590
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152460958
LGTM except the small styling issues.
---
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 mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474244
--- Diff:
examples/src/main/java/org/apache/spark/examples/mllib/JavaNaiveBayesExample.java
---
@@ -0,0 +1,68 @@
+/*
+ * Licensed to the Apache
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474236
--- Diff:
examples/src/main/java/org/apache/spark/examples/mllib/JavaIsotonicRegressionExample.java
---
@@ -0,0 +1,89 @@
+/*
+ * Licensed to the
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474257
--- Diff: examples/src/main/python/mllib/naive_bayes_example.py ---
@@ -0,0 +1,56 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474262
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/IsotonicRegressionExample.scala
---
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474241
--- Diff:
examples/src/main/java/org/apache/spark/examples/mllib/JavaIsotonicRegressionExample.java
---
@@ -0,0 +1,89 @@
+/*
+ * Licensed to the
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474260
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/IsotonicRegressionExample.scala
---
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474263
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/IsotonicRegressionExample.scala
---
@@ -0,0 +1,66 @@
+/*
+ * Licensed to the Apache
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474242
--- Diff:
examples/src/main/java/org/apache/spark/examples/mllib/JavaIsotonicRegressionExample.java
---
@@ -0,0 +1,89 @@
+/*
+ * Licensed to the
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474253
--- Diff: examples/src/main/python/mllib/isotonic_regression_example.py ---
@@ -0,0 +1,56 @@
+#
+# Licensed to the Apache Software Foundation (ASF)
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474240
--- Diff:
examples/src/main/java/org/apache/spark/examples/mllib/JavaIsotonicRegressionExample.java
---
@@ -0,0 +1,89 @@
+/*
+ * Licensed to the
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474245
--- Diff:
examples/src/main/java/org/apache/spark/examples/mllib/JavaNaiveBayesExample.java
---
@@ -0,0 +1,68 @@
+/*
+ * Licensed to the Apache
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474248
--- Diff:
examples/src/main/java/org/apache/spark/examples/mllib/JavaNaiveBayesExample.java
---
@@ -0,0 +1,68 @@
+/*
+ * Licensed to the Apache
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/9353#discussion_r43474266
--- Diff:
examples/src/main/scala/org/apache/spark/examples/mllib/NaiveBayesExample.scala
---
@@ -0,0 +1,57 @@
+/*
+ * Licensed to the Apache
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9373#issuecomment-152441415
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
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/9373#discussion_r43475449
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/util/FileBasedWriteAheadLog.scala
---
@@ -149,27 +150,26 @@ private[streaming] class
Github user WangTaoTheTonic commented on the pull request:
https://github.com/apache/spark/pull/5664#issuecomment-152442824
@jacek-lewandowski Okay I will rebase this in a week or so.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/9373#discussion_r43475406
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/JobScheduler.scala
---
@@ -88,8 +88,10 @@ class JobScheduler(val ssc:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9353#issuecomment-152446497
**[Test build #44668 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44668/consoleFull)**
for PR 9353 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9353#issuecomment-152449251
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9353#issuecomment-152449148
**[Test build #44668 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44668/consoleFull)**
for PR 9353 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9353#issuecomment-152449252
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9327#issuecomment-152452492
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
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9232#discussion_r43478397
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala ---
@@ -142,6 +145,81 @@ class YarnSparkHadoopUtil extends
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152457453
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152457370
**[Test build #44669 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44669/consoleFull)**
for PR 9180 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152457451
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
GitHub user brkyvz opened a pull request:
https://github.com/apache/spark/pull/9373
[SPARK-11419][STREAMING] Par recovery
The support for closing WriteAheadLog files after writes was just merged
in. Closing every file after a write is a very expensive operation as it
creates many
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152446195
Merged build finished. Test FAILed.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9353#issuecomment-152446187
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/9343#discussion_r43477553
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
---
@@ -166,26 +168,55 @@ abstract class Star extends
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9313#issuecomment-152452265
**[Test build #44670 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44670/consoleFull)**
for PR 9313 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9232#discussion_r43478370
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -1337,55 +1337,9 @@ object Client extends Logging {
conf:
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9327#issuecomment-152452422
**[Test build #44664 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44664/consoleFull)**
for PR 9327 at commit
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9180#discussion_r43481002
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala
---
@@ -49,16 +50,29 @@ class LinearRegressionSuite extends
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152445929
**[Test build #44667 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44667/consoleFull)**
for PR 9180 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9348#issuecomment-152448549
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9345#issuecomment-152450893
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/9359#discussion_r43477839
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -35,7 +35,8 @@ class DatasetSuite extends QueryTest with
SharedSQLContext
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9345#issuecomment-152450896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8587#issuecomment-152453486
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8587#issuecomment-152453498
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
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/9180#discussion_r43481079
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/LinearRegressionSuite.scala
---
@@ -49,16 +50,29 @@ class LinearRegressionSuite extends
Github user jbonofre commented on a diff in the pull request:
https://github.com/apache/spark/pull/9165#discussion_r43474394
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/ExecutorTable.scala
---
@@ -102,7 +102,7 @@ private[ui] class ExecutorTable(stageId: Int,
Github user Lewuathe commented on a diff in the pull request:
https://github.com/apache/spark/pull/9180#discussion_r43474383
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/util/LinearDataGenerator.scala ---
@@ -112,8 +139,13 @@ object LinearDataGenerator {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9373#issuecomment-152442342
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9373#issuecomment-152442357
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8587#issuecomment-152444874
**[Test build #44659 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44659/consoleFull)**
for PR 8587 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8587#issuecomment-152444984
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8587#issuecomment-152444982
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152450697
**[Test build #44669 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44669/consoleFull)**
for PR 9180 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9345#issuecomment-152450742
**[Test build #44663 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44663/consoleFull)**
for PR 9345 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9232#discussion_r43478531
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtil.scala ---
@@ -142,6 +145,81 @@ class YarnSparkHadoopUtil extends
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9232#discussion_r43478556
--- Diff:
yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtilSuite.scala
---
@@ -245,4 +247,31 @@ class YarnSparkHadoopUtilSuite extends
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152462600
**[Test build #44672 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44672/consoleFull)**
for PR 9206 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9373#issuecomment-152441892
**[Test build #44665 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44665/consoleFull)**
for PR 9373 at commit
Github user JihongMA closed the pull request at:
https://github.com/apache/spark/pull/8778
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9373#issuecomment-152441896
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9373#issuecomment-152441894
Merged build finished. Test FAILed.
---
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
Github user JihongMA commented on the pull request:
https://github.com/apache/spark/pull/8778#issuecomment-152441928
closing this pull request. based on the performance comparison of
Declartive vs. Imperative aggregate (SPARK-10953), the implementation of
central moment computation
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152446197
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152446191
**[Test build #44667 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44667/consoleFull)**
for PR 9180 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9353#issuecomment-152446165
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8587#issuecomment-152453875
**[Test build #44671 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44671/consoleFull)**
for PR 8587 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152461532
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
Github user sun-rui commented on the pull request:
https://github.com/apache/spark/pull/8984#issuecomment-152465285
Per the Scala API doc on cast() of Column, the supported types are: string,
boolean, byte, short, int, long, float, double, decimal, date, timestamp. That
is, complext
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9372#issuecomment-152448936
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9372#issuecomment-152448937
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152449029
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9372#issuecomment-152448832
**[Test build #44660 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44660/consoleFull)**
for PR 9372 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9180#issuecomment-152449043
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
Github user rishabhbhardwaj commented on the pull request:
https://github.com/apache/spark/pull/9353#issuecomment-152450116
@mengxr Thanks for your comments and patience.I have made the required
changes.
---
If your project is set up for it, you can reply to this email and have your
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9232#discussion_r43478640
--- Diff:
yarn/src/test/scala/org/apache/spark/deploy/yarn/YarnSparkHadoopUtilSuite.scala
---
@@ -245,4 +247,28 @@ class YarnSparkHadoopUtilSuite extends
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/9232#issuecomment-152453057
Just some minor things left to clean up, otherwise looks ok.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/9182#discussion_r43479330
--- Diff:
yarn/src/main/scala/org/apache/spark/scheduler/cluster/YarnSchedulerBackend.scala
---
@@ -51,6 +51,41 @@ private[spark] abstract class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9206#issuecomment-152461553
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
1 - 100 of 655 matches
Mail list logo