Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/15654#discussion_r85799941
--- Diff:
mesos/src/main/scala/org/apache/spark/deploy/mesos/MesosClusterDispatcher.scala
---
@@ -51,7 +52,7 @@ private[mesos] class
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/15654
Thanks! One small fix then LGTM
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15628
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67819/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15628
**[Test build #67819 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67819/consoleFull)**
for PR 15628 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15628
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 project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15659
**[Test build #67814 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67814/consoleFull)**
for PR 15659 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15693
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67811/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15693
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15693
**[Test build #67811 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67811/consoleFull)**
for PR 15693 at commit
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/13713
We need to get @srowen or one of the other committers to merge 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
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/15651
lgtm pending jenkins
---
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,
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/1
ok to test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15626
**[Test build #67822 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67822/consoleFull)**
for PR 15626 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15654
**[Test build #67823 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67823/consoleFull)**
for PR 15654 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15651
**[Test build #67816 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67816/consoleFull)**
for PR 15651 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15627
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67826/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15627
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15627
**[Test build #67826 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67826/consoleFull)**
for PR 15627 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15651
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67816/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15651
**[Test build #67817 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67817/consoleFull)**
for PR 15651 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15651
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67817/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15651
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
GitHub user JoshRosen opened a pull request:
https://github.com/apache/spark/pull/15698
[SPARK-18182] Expose ReplayListenerBus.read() overload which takes string
iterator
The `ReplayListenerBus.read()` method is used when implementing a custom
`ApplicationHistoryProvider`. The
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15651
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15698
**[Test build #67828 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67828/consoleFull)**
for PR 15698 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/11105
ping @squito / @rxin if either of you have some post-Spark Summit EU
bandwidth to review this it would be awesome :)
---
If your project is set up for it, you can reply to this email and have your
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15659
**[Test build #67814 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67814/consoleFull)**
for PR 15659 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15697
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67827/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15697
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15659
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67814/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15697
**[Test build #67827 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67827/consoleFull)**
for PR 15697 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15659
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/15538
Are we planning to incorporate the Parquet 1.9 libraries into Spark 2.1? If
so, then this PR should be unnecessary.
Hopefully.
---
If your project is set up for it, you can reply to this
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/15654
cc @srowen for merge into 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 this feature
enabled
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/14803#discussion_r85819725
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSourceSuite.scala
---
@@ -608,6 +614,81 @@ class FileStreamSourceSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15626
**[Test build #67822 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67822/consoleFull)**
for PR 15626 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15626
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67822/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15626
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 project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15696
**[Test build #67818 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67818/consoleFull)**
for PR 15696 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15696
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15696
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67818/
Test PASSed.
---
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/13881
Sorry for the long delay! Whenever you get a chance to update this, it'd
be nice to log this info via the Instrumentation class, rather than logInfo.
---
If your project is set up for it, you
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15659
**[Test build #67824 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67824/consoleFull)**
for PR 15659 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11105
**[Test build #67825 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67825/consoleFull)**
for PR 11105 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15654
**[Test build #67823 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67823/consoleFull)**
for PR 15654 at commit
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15667#discussion_r85808139
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala
---
@@ -257,7 +258,31 @@ case class InsertIntoHiveTable(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15024#discussion_r85808919
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -418,21 +424,41 @@ private[spark] class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15695
**[Test build #67815 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67815/consoleFull)**
for PR 15695 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15627
**[Test build #67826 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67826/consoleFull)**
for PR 15627 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15697
**[Test build #67827 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67827/consoleFull)**
for PR 15697 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15637
How about the complex types? `Array`, `Map` and `Struct`? It sounds like
the test cases do not cover these test cases. Thanks!
---
If your project is set up for it, you can reply to this email
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15651
**[Test build #67813 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67813/consoleFull)**
for PR 15651 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15637#discussion_r85814522
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/MapAggregate.scala
---
@@ -0,0 +1,332 @@
+/*
+ *
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/11105#discussion_r85805218
--- Diff: core/src/main/scala/org/apache/spark/util/JsonProtocol.scala ---
@@ -722,6 +722,7 @@ private[spark] object JsonProtocol {
val value =
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15695
**[Test build #67812 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67812/consoleFull)**
for PR 15695 at commit
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/15692
You'll need to add the Param itself. (Search for ```Params.dummy()``` in
that file to find examples.)
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15695
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 project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15695
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67812/
Test FAILed.
---
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15633#discussion_r85806652
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -179,24 +180,30 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15654
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67823/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15654
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user techaddict commented on the issue:
https://github.com/apache/spark/pull/15654
@mgummelt done! ð
---
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
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15667#discussion_r85807729
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala
---
@@ -257,7 +258,31 @@ case class InsertIntoHiveTable(
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/15538#discussion_r85809383
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFileFormat.scala
---
@@ -55,6 +56,21 @@ class ParquetFileFormat
GitHub user wangmiao1981 opened a pull request:
https://github.com/apache/spark/pull/15697
[SparkR][Test]:remove unnecessary suppressWarnings
## What changes were proposed in this pull request?
In test_mllib.R, there are two unnecessary suppressWarnings. This PR just
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15695
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 project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15695
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67815/
Test FAILed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15637#discussion_r85811315
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/MapAggregateQuerySuite.scala ---
@@ -0,0 +1,144 @@
+/*
+ * Licensed to the Apache Software
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15695
Looks like FileStreamSourceSuite is broken is 2.0. Looking at 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15651
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15651
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67813/
Test PASSed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15637#discussion_r85812423
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/MapAggregate.scala
---
@@ -0,0 +1,332 @@
+/*
+ *
501 - 572 of 572 matches
Mail list logo