Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/18101
Thanks. Merging to master, 2.2 and 2.1.
---
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
ena
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/18051
That makes sense!
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18101
---
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 ena
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18094
**[Test build #77379 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77379/testReport)**
for PR 18094 at commit
[`a2f99ec`](https://github.com/apache/spark/commit/a2
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18105
If users have not analyzed the table in Spark yet, we should respect the
stats from hive metastore. But if users have already run the analyze table
command in Spark, I think it's fair to ask them
GitHub user mallman opened a pull request:
https://github.com/apache/spark/pull/18112
[SPARK-20888][SQL][DOCS] Document change of default setting of
spark.sql.hive.caseSensitiveInferenceMode
(Link to Jira: https://issues.apache.org/jira/browse/SPARK-20888)
## What changes w
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18094
**[Test build #77379 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77379/testReport)**
for PR 18094 at commit
[`a2f99ec`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18094
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77379/
Test FAILed.
---
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/11746#discussion_r118544204
--- Diff:
core/src/main/scala/org/apache/spark/deploy/worker/DriverRunner.scala ---
@@ -53,9 +53,11 @@ private[deploy] class DriverRunner(
@volatile
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18094
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
e
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/18112
@budde Can you please review (urgently) for inclusion as a migration note
for 2.2?
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18112
**[Test build #77380 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77380/testReport)**
for PR 18112 at commit
[`037e7d3`](https://github.com/apache/spark/commit/03
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18094
**[Test build #77381 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77381/testReport)**
for PR 18094 at commit
[`27a8c26`](https://github.com/apache/spark/commit/27
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/18092
>> I can not think of meaningful test cases, are there any suggestions?
How about just "unit tests" ?
---
If your project is set up for it, you can reply to this email and have your
repl
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/18110
@cloud-fan, what about `SparkConf`'s `configsWithAlternatives `:
https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/SparkConf.scala#L596
---
If your project is set
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11974#discussion_r118546602
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/clustering/KMeansSuite.scala ---
@@ -89,6 +92,9 @@ class KMeansSuite extends SparkFunSuite with
MLlibTest
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/17471
@sameeragarwal - Thanks for taking a look. I will update the PR adding test
case soon.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18110
It's only used in the `SparkCong.get(key: String)` code path, not
`SparkConf.get(entry: ConfigEntry[T])` code path. That's why we only support
alternative keys if users get conf value by hard-code
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/18092#discussion_r118547246
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -170,11 +170,17 @@ private[spark] class BlockManager(
// service,
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/16578
Also, I'm confused about somethingâwho has jenkins retest privileges? And
can I get them?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18112
**[Test build #77380 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77380/testReport)**
for PR 18112 at commit
[`037e7d3`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18112
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77380/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18112
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
e
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11974#discussion_r118548583
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -85,6 +85,20 @@ private[clustering] trait KMeansParams extends Params
with
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/18110
Ahhh, makes sense. Thanks for the clarification.
---
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 MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11974#discussion_r118548684
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -85,6 +85,20 @@ private[clustering] trait KMeansParams extends Params
with
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/11974#discussion_r118548792
--- Diff: mllib/src/main/scala/org/apache/spark/ml/clustering/KMeans.scala
---
@@ -85,6 +85,20 @@ private[clustering] trait KMeansParams extends Params
with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18064
**[Test build #77374 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77374/testReport)**
for PR 18064 at commit
[`def0878`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18064
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77374/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18064
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
e
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/11974
cc @srowen @setha also
---
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 zero323 commented on the issue:
https://github.com/apache/spark/pull/18051
Exactly my point. Run examples internally ([it is not hard to patch
knitr](https://github.com/zero323/knitr/commit/7a0d8f9ddb9d77a9c235f25aca26131e83c1f6cc)
or even `tools::Rd2ex`) to validate examp
Github user zero323 closed the pull request at:
https://github.com/apache/spark/pull/18051
---
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 en
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/18112
CC @cloud-fan @ericl
---
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 JoshRosen commented on the issue:
https://github.com/apache/spark/pull/18083
> I am not sure that monitoring (with real metrics) the number of dropped
events really worth it. You just want to know if messages have been dropped
(and having the number in the log is fine).
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/18098#discussion_r118550928
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -393,7 +409,30 @
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17094
**[Test build #77376 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77376/testReport)**
for PR 17094 at commit
[`29052d3`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17094
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77376/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17094
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17343
**[Test build #77382 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77382/testReport)**
for PR 17343 at commit
[`d4f09c2`](https://github.com/apache/spark/commit/d4
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/17864
Originally the idea behind only supporting double was as @sethah posted
above - there could be some issues with handling of int casting etc. As
mentioned originally, we did consider "always cast to d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18064
**[Test build #77383 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77383/testReport)**
for PR 18064 at commit
[`59ec0e9`](https://github.com/apache/spark/commit/59
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18025
**[Test build #77378 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77378/testReport)**
for PR 18025 at commit
[`21e54c0`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18025
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77378/
Test PASSed.
---
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/17343#discussion_r118553894
--- Diff:
core/src/main/java/org/apache/spark/shuffle/sort/UnsafeShuffleWriter.java ---
@@ -339,23 +355,26 @@ void forceSorterToSpill() throws IOExceptio
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/17343
Thanks @sameeragarwal, addressed the check style issues. Yes, the exisiting
unit tests in UnsafeShuffleWriter#mergeSpillsWithTransferToAndLZF covers this
code.
---
If your project is set up fo
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18064
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77383/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18064
**[Test build #77383 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77383/testReport)**
for PR 18064 at commit
[`59ec0e9`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18064
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
e
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/18064
@cloud-fan, can you summarize how this differs from the original PR #17540?
I have time to pick this up again, but I thought that the other PR only
needed two changes:
* Merge your c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18110
**[Test build #77372 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77372/testReport)**
for PR 18110 at commit
[`cc51dd0`](https://github.com/apache/spark/commit/c
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/18065#discussion_r118555796
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/DataStreamReader.scala
---
@@ -35,7 +35,6 @@ import org.apache.spark.sql.types.StructType
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18110
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18110
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77372/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18111
**[Test build #77373 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77373/testReport)**
for PR 18111 at commit
[`02eb7bf`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18111
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18111
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77373/
Test PASSed.
---
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/18098#discussion_r118558695
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -393,7 +410,30 @
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18014
**[Test build #77384 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77384/testReport)**
for PR 18014 at commit
[`bf6ab20`](https://github.com/apache/spark/commit/bf
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18098
**[Test build #77375 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77375/testReport)**
for PR 18098 at commit
[`3e656e5`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18098
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18098
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77375/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18014
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18014
**[Test build #77384 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77384/testReport)**
for PR 18014 at commit
[`bf6ab20`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18014
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77384/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18097
**[Test build #77377 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77377/testReport)**
for PR 18097 at commit
[`90433a9`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18097
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77377/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18097
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18014
**[Test build #77385 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77385/testReport)**
for PR 18014 at commit
[`9954d6b`](https://github.com/apache/spark/commit/99
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18107
**[Test build #77386 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77386/testReport)**
for PR 18107 at commit
[`324fc24`](https://github.com/apache/spark/commit/32
Github user dhruve commented on a diff in the pull request:
https://github.com/apache/spark/pull/14269#discussion_r118573078
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/LiveListenerBus.scala ---
@@ -32,18 +33,24 @@ import org.apache.spark.util.Utils
* has started
Github user dhruve commented on a diff in the pull request:
https://github.com/apache/spark/pull/14269#discussion_r118573206
--- Diff:
core/src/main/scala/org/apache/spark/internal/config/package.scala ---
@@ -103,4 +103,9 @@ package object config {
.stringConf
.c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18094
**[Test build #77381 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77381/testReport)**
for PR 18094 at commit
[`27a8c26`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18094
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18094
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77381/
Test PASSed.
---
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/11974
Mini-batching in Spark generally isn't that efficient, since to extract a
mini-batch you still need to iterate over the entire dataset - and that means
reading it from disk if it doesn't fit into mem
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18078#discussion_r118580623
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkSubmitSuite.scala ---
@@ -535,7 +538,7 @@ class SparkSubmitSuite
test("resolves
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18078#discussion_r118580006
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -308,6 +311,15 @@ object SparkSubmit extends CommandLineUtils {
Github user gpang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18098#discussion_r118581400
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -393,7 +409,30 @@ p
Github user gpang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18098#discussion_r118581450
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -393,7 +410,30 @@ p
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18078
Could you also add "[Core]" tag in the title? @loneknightpy
Also cc @cloud-fan @gatorsmile
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18098
**[Test build #77387 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77387/testReport)**
for PR 18098 at commit
[`fa2daff`](https://github.com/apache/spark/commit/fa
Github user loneknightpy commented on a diff in the pull request:
https://github.com/apache/spark/pull/18078#discussion_r118583624
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkSubmitSuite.scala ---
@@ -535,7 +538,7 @@ class SparkSubmitSuite
test("resolves
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/18098#discussion_r118586090
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -502,6 +521,25 @
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17343
**[Test build #77382 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77382/testReport)**
for PR 17343 at commit
[`d4f09c2`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17343
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17343
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77382/
Test PASSed.
---
GitHub user setjet opened a pull request:
https://github.com/apache/spark/pull/18113
[SPARK-20890][SQL] Added min and max typed aggregation functions
## What changes were proposed in this pull request?
Typed Min and Max functions are missing for aggregations done on dataset.
The
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18113
Can one of the admins verify this patch?
---
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 feat
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18078
**[Test build #77388 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77388/testReport)**
for PR 18078 at commit
[`e5171ca`](https://github.com/apache/spark/commit/e5
Github user gpang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18098#discussion_r118590493
--- Diff:
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosCoarseGrainedSchedulerBackend.scala
---
@@ -502,6 +521,25 @@ p
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18078
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 and wishes so, or if the f
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18078#discussion_r118593252
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkSubmitSuite.scala ---
@@ -535,7 +538,7 @@ class SparkSubmitSuite
test("resolves
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18078
**[Test build #77389 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77389/testReport)**
for PR 18078 at commit
[`62e57df`](https://github.com/apache/spark/commit/62
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/18107#discussion_r118595928
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FlatMapGroupsWithStateSuite.scala
---
@@ -508,22 +508,6 @@ class FlatMapGroupsWithStateSuite
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18014
**[Test build #77385 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77385/testReport)**
for PR 18014 at commit
[`9954d6b`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18014
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18014
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77385/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18107
**[Test build #77386 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77386/testReport)**
for PR 18107 at commit
[`324fc24`](https://github.com/apache/spark/commit/3
301 - 400 of 584 matches
Mail list logo