Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10266#issuecomment-163903552
**[Test build #47578 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47578/consoleFull)**
for PR 10266 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10266#issuecomment-163903666
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/10266#issuecomment-163903664
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 chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/10265#issuecomment-163886140
LGTM
cc/ @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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10174#issuecomment-163889311
**[Test build #47577 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47577/consoleFull)**
for PR 10174 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9957#issuecomment-163919024
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/9957#issuecomment-163919026
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/8880#issuecomment-163903501
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/8880#issuecomment-163903504
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/8880#issuecomment-163903431
**[Test build #47579 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47579/consoleFull)**
for PR 8880 at commit
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/9957#issuecomment-163910194
Jenkins, 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
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/9158#issuecomment-163909768
Jenkins, 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10174#issuecomment-163889668
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/10174#issuecomment-163889666
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/9957#issuecomment-163918742
**[Test build #47581 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47581/consoleFull)**
for PR 9957 at commit
Github user liancheng commented on a diff in the pull request:
https://github.com/apache/spark/pull/10215#discussion_r47350129
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala ---
@@ -161,17 +161,15 @@ class DataFrame private[sql](
}
/**
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9158#issuecomment-163912865
[Test build #47582 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47582/consoleFull)
for PR 9158 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9957#issuecomment-163913829
**[Test build #47581 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47581/consoleFull)**
for PR 9957 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10201#issuecomment-163887087
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/10201#issuecomment-163887085
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/10201#issuecomment-163886938
**[Test build #47580 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47580/consoleFull)**
for PR 10201 at commit
Github user dragos commented on the pull request:
https://github.com/apache/spark/pull/8872#issuecomment-163901420
I'd love to move this forward, but I need to know how you tested this (and
replicate). ping @tnachen
---
If your project is set up for it, you can reply to this email
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/10266#discussion_r47351058
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala
---
@@ -1029,24 +1029,27 @@ case class ScalaUDF(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9158#issuecomment-163945472
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/9158#issuecomment-163945470
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 does
Github user dragos commented on the pull request:
https://github.com/apache/spark/pull/4027#issuecomment-163952266
@tnachen @andrewor14 what should be the way forward? I know about a more
ambitious plan to rewrite the Mesos scheduler. Should we still push this in the
interim, or just
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10256#issuecomment-163945864
**[Test build #2203 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2203/consoleFull)**
for PR 10256 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10256#discussion_r47350412
--- Diff: pom.xml ---
@@ -169,7 +169,7 @@
${scala.version}
org.scala-lang
1.9.13
-2.4.4
+2.5.3
--- End diff --
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10256#issuecomment-163927639
**[Test build #2203 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2203/consoleFull)**
for PR 10256 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9158#issuecomment-163945347
[Test build #47582 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47582/console)
for PR 9158 at commit
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/10266#issuecomment-163983570
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10266#issuecomment-163984799
**[Test build #2204 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2204/consoleFull)**
for PR 10266 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10231#issuecomment-163987356
**[Test build #47583 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47583/consoleFull)**
for PR 10231 at commit
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/10252#discussion_r47366152
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/dstream/DStream.scala ---
@@ -330,6 +330,20 @@ abstract class DStream[T: ClassTag] (
Github user markhamstra commented on the pull request:
https://github.com/apache/spark/pull/10266#issuecomment-163972430
Works for me. Thanks, guys!
---
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
Github user BrianLondon commented on the pull request:
https://github.com/apache/spark/pull/10256#issuecomment-163985429
@brkyvz Pushed up the KCL version and added a newline to KinesisReciever.
Tests passed for me locally on a hadoop 2.6 build and the kinesis demos ran
against a
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10266#discussion_r47374514
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala
---
@@ -1029,24 +1029,27 @@ case class ScalaUDF(
//
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10266#issuecomment-163986683
**[Test build #47584 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47584/consoleFull)**
for PR 10266 at commit
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/10252#discussion_r47364512
--- Diff: streaming/src/test/scala/org/apache/spark/streaming/Demo.scala ---
@@ -0,0 +1,94 @@
+/*
--- End diff --
Can this be a unit
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/10252#discussion_r47365574
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/dstream/DSteamDumpFormat.scala
---
@@ -0,0 +1,59 @@
+/*
+ * Licensed to the
Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/10252#discussion_r47364366
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/dstream/StateDStream.scala
---
@@ -22,8 +22,10 @@ import org.apache.spark.Partitioner
Github user JasonMWhite commented on a diff in the pull request:
https://github.com/apache/spark/pull/10089#discussion_r47363415
--- Diff:
external/kafka/src/test/scala/org/apache/spark/streaming/kafka/DirectKafkaStreamSuite.scala
---
@@ -364,8 +365,8 @@ class
Github user jbonofre commented on the pull request:
https://github.com/apache/spark/pull/10203#issuecomment-164053350
Thanks @andrewor14 (again ;)). Let me retest it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9819#discussion_r47408969
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/windowExpressions.scala
---
@@ -246,85 +260,244 @@ object
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10265#issuecomment-164054314
**[Test build #47592 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47592/consoleFull)**
for PR 10265 at commit
Github user jbonofre commented on the pull request:
https://github.com/apache/spark/pull/10203#issuecomment-164057641
Tests OK on my box. The Jenkins test failure doesn't look related.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/10215#discussion_r47409690
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrame.scala ---
@@ -161,17 +161,15 @@ class DataFrame private[sql](
}
/**
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5423#issuecomment-164061802
**[Test build #47596 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47596/consoleFull)**
for PR 5423 at commit
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/9108#issuecomment-164068109
I'd like to get this in (and have time now), so could you please fix the
merge issues when you have a chance? Thank you!
---
If your project is set up for it, you
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/9108#discussion_r47414442
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tuning/CrossValidator.scala ---
@@ -45,6 +46,9 @@ private[ml] trait CrossValidatorParams extends
Github user lmtjalves commented on the pull request:
https://github.com/apache/spark/pull/9301#issuecomment-164053007
Ok I'll add the placeholder. The idea of this PR was mostly to make the
current dockerfile work (or at least make more sence).
---
If your project is set up for it,
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9819#discussion_r47407972
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/windowExpressions.scala
---
@@ -246,85 +260,244 @@ object
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/9819#discussion_r47409455
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Window.scala ---
@@ -736,15 +691,156 @@ private[execution] final class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9819#discussion_r47410151
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Window.scala ---
@@ -736,15 +691,156 @@ private[execution] final class
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/10257#issuecomment-164067304
LGTM pending tests
Thanks!
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10268#issuecomment-164069485
**[Test build #2210 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2210/consoleFull)**
for PR 10268 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10130#issuecomment-164084957
**[Test build #47594 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47594/consoleFull)**
for PR 10130 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10130#issuecomment-164085087
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/10130#issuecomment-164085089
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/9819#discussion_r47406503
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/windowExpressions.scala
---
@@ -120,6 +121,19 @@ sealed trait FrameBoundary {
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/10265#issuecomment-164051963
ok to test. @yhuai can you verify?
---
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/10264#issuecomment-164053802
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/10264#issuecomment-164053804
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user BenFradet commented on the pull request:
https://github.com/apache/spark/pull/10257#issuecomment-164066490
@jkbradley thanks for the 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10257#issuecomment-164067852
**[Test build #2209 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2209/consoleFull)**
for PR 10257 at commit
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/9108#issuecomment-164069109
That should be it, thanks
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/9458
---
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/10257#issuecomment-164070917
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/10257#issuecomment-164070895
**[Test build #2209 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2209/consoleFull)**
for PR 10257 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10257#issuecomment-164070919
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/10268#issuecomment-164073024
**[Test build #2210 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2210/consoleFull)**
for PR 10268 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8744#issuecomment-164083850
**[Test build #47595 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47595/consoleFull)**
for PR 8744 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8744#issuecomment-164084088
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user chenghao-intel commented on the pull request:
https://github.com/apache/spark/pull/10265#issuecomment-164086130
The failure seems not related to this change.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/9819#discussion_r47406421
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -70,15 +70,32 @@ trait CheckAnalysis {
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8872#discussion_r47407104
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosClusterScheduler.scala
---
@@ -358,9 +358,10 @@ private[spark] class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9819#discussion_r47407165
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/CheckAnalysis.scala
---
@@ -70,15 +70,32 @@ trait CheckAnalysis {
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10268#issuecomment-164051335
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/10268#issuecomment-164051299
**[Test build #47591 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47591/consoleFull)**
for PR 10268 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10268#issuecomment-164051339
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user jbonofre commented on the pull request:
https://github.com/apache/spark/pull/10130#issuecomment-164053126
Thanks for the update @andrewor14. It makes sense for the new functions().
Exposing the showString() is enough for most of the cases. Let me update the PR.
---
If
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10179
---
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 hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9819#discussion_r47408697
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -870,26 +878,37 @@ class Analyzer(
//
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8744#issuecomment-164059610
**[Test build #47595 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47595/consoleFull)**
for PR 8744 at commit
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/9819#discussion_r47410028
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Window.scala ---
@@ -149,43 +152,102 @@ case class Window(
}
/**
-
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/10267#issuecomment-164059133
@viirya @zsxwing what is the root cause of this issue? It seems arbitrary
to just cut off an attempt if the pool size is already at the capacity. I would
think that
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10261#issuecomment-164059104
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/10261#issuecomment-164059097
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/10269#issuecomment-164060902
**[Test build #47593 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47593/consoleFull)**
for PR 10269 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9819#discussion_r47411273
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/Window.scala ---
@@ -736,15 +691,156 @@ private[execution] final class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10215#issuecomment-164062523
**[Test build #47597 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47597/consoleFull)**
for PR 10215 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10269#issuecomment-164071523
**[Test build #47602 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47602/consoleFull)**
for PR 10269 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10203#issuecomment-164071440
**[Test build #2208 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2208/consoleFull)**
for PR 10203 at commit
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/10235#issuecomment-164080820
CC @thunterdb Could you please check this? The content should be OK since
@mengxr verified it previously.
---
If your project is set up for it, you can reply to
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/10257#issuecomment-164081063
Merging with master and branch-1.6
---
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 proflin commented on the pull request:
https://github.com/apache/spark/pull/10264#issuecomment-164087083
Thanks @zsxwing for reviewing so soon :-)
---
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 andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/8872#discussion_r47406920
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosClusterScheduler.scala
---
@@ -358,9 +358,10 @@ private[spark] class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/9819#discussion_r47408553
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/interfaces.scala
---
@@ -187,7 +184,7 @@ sealed abstract class
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10264#issuecomment-164053664
**[Test build #47590 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47590/consoleFull)**
for PR 10264 at commit
Github user RotemShaul commented on the pull request:
https://github.com/apache/spark/pull/10190#issuecomment-164053452
Any news ? Did anyone look into it ?
---
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
1 - 100 of 337 matches
Mail list logo
