Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/15324
Jenkins, retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled a
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82733349
--- Diff:
core/src/main/java/org/apache/spark/io/NioBufferedFileInputStream.java ---
@@ -0,0 +1,129 @@
+/*
+ * Licensed under the Apache License, Vers
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15377
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66719/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15377
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/15377
**[Test build #66719 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66719/consoleFull)**
for PR 15377 at commit
[`df28bdd`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15285
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/15285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66718/
Test PASSed.
---
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82732392
--- Diff:
core/src/main/java/org/apache/spark/io/NioBufferedFileInputStream.java ---
@@ -0,0 +1,129 @@
+/*
+ * Licensed under the Apache License, Versio
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15285
**[Test build #66718 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66718/consoleFull)**
for PR 15285 at commit
[`e5676a6`](https://github.com/apache/spark/commit/
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82732276
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,127 @@
+/*
+ * Licensed under the Apache License, V
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15360#discussion_r82731979
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala ---
@@ -358,50 +358,180 @@ class StatisticsSuite extends QueryTest with
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15408
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66714/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15408
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/15408
**[Test build #66714 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66714/consoleFull)**
for PR 15408 at commit
[`681ff62`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15377
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66715/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15377
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15360#discussion_r82731661
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeColumnCommand.scala
---
@@ -62,7 +62,7 @@ case class AnalyzeColumnCommand
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15377
**[Test build #66715 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66715/consoleFull)**
for PR 15377 at commit
[`7485ffa`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11459
**[Test build #66726 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66726/consoleFull)**
for PR 11459 at commit
[`ab05aa6`](https://github.com/apache/spark/commit/a
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15360#discussion_r82730881
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala ---
@@ -358,50 +358,180 @@ class StatisticsSuite extends QueryTest with
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15386
---
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 srowen commented on the issue:
https://github.com/apache/spark/pull/15386
Merged to master
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or i
Github user SaintBacchus commented on a diff in the pull request:
https://github.com/apache/spark/pull/15297#discussion_r82730696
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -138,13 +138,16 @@ private[spark] abstract class MapOutputTracker(conf:
Spa
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15285
**[Test build #66725 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66725/consoleFull)**
for PR 15285 at commit
[`7cc6935`](https://github.com/apache/spark/commit/7
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15360#discussion_r82730527
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala ---
@@ -358,50 +358,180 @@ class StatisticsSuite extends QueryTest with
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15072#discussion_r82730295
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -53,7 +53,15 @@ import org.apache.spark.util.Utils
private[sql] obj
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15427
**[Test build #66724 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66724/consoleFull)**
for PR 15427 at commit
[`dd6405c`](https://github.com/apache/spark/commit/d
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/15427
[SPARK-17866][SPARK-17867][SQL] Fix Dataset.dropduplicates
## What changes were proposed in this pull request?
Two issues regarding Dataset.dropduplicates:
1. Dataset.dropDuplicates
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15285
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15285
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66716/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15285
**[Test build #66716 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66716/consoleFull)**
for PR 15285 at commit
[`ef4f2b9`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14702
**[Test build #66723 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66723/consoleFull)**
for PR 14702 at commit
[`c7741f9`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15295
**[Test build #66722 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66722/consoleFull)**
for PR 15295 at commit
[`0ad8815`](https://github.com/apache/spark/commit/0
Github user SaintBacchus commented on a diff in the pull request:
https://github.com/apache/spark/pull/15297#discussion_r82728585
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SkewShuffleRowRDD.scala
---
@@ -0,0 +1,147 @@
+/*
+ * Licensed to the Apache S
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/15072#discussion_r82728542
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -53,7 +53,15 @@ import org.apache.spark.util.Utils
private[sql] obje
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14788
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66713/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14788
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14702
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14788
**[Test build #66713 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66713/consoleFull)**
for PR 14788 at commit
[`ef67829`](https://github.com/apache/spark/commit/
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/15072#discussion_r82728292
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -53,7 +53,15 @@ import org.apache.spark.util.Utils
private[sql] obj
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13675
---
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 yanboliang commented on the issue:
https://github.com/apache/spark/pull/13675
I'll merge this into master, thanks for review! @jkbradley @felixcheung
---
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 pr
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/13675
@felixcheung This PR does not affect R code, I will send another PR to fix
issues like [SPARK-15153](https://issues.apache.org/jira/browse/SPARK-15153)
which need to add some R tests.
---
If yo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15425
**[Test build #3321 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3321/consoleFull)**
for PR 15425 at commit
[`678ee6b`](https://github.com/apache/spark/commit/
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15425
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 have this feature
enabled and wishes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15295
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15295
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66706/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15295
**[Test build #66706 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66706/consoleFull)**
for PR 15295 at commit
[`8d93c4a`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15424
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/15424
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66707/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15424
**[Test build #66707 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66707/consoleFull)**
for PR 15424 at commit
[`15efca6`](https://github.com/apache/spark/commit/
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15292
@gatorsmile @cloud-fan Thank you for reviewing this both!
---
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 no
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15388
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66708/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15388
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/15412
---
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/15388
**[Test build #66708 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66708/consoleFull)**
for PR 15388 at commit
[`21958d7`](https://github.com/apache/spark/commit/
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/15412
LGTM - merging to master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wi
Github user yangw1234 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15416#discussion_r82726593
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -298,10 +298,14 @@ class Analyzer(
case
Github user Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r82726587
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,343 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) unde
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15292
Thanks! Merging to master!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and w
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/15408#discussion_r82726120
--- Diff:
core/src/main/java/org/apache/spark/io/NioBasedBufferedFileInputStream.java ---
@@ -0,0 +1,127 @@
+/*
+ * Licensed under the Apache License,
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/15072#discussion_r82726273
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -53,7 +53,15 @@ import org.apache.spark.util.Utils
private[sql] obje
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15292
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15412
cc @hvanhovell ?
---
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r82725489
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,343 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) und
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15148
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66717/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15148
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/15148
**[Test build #66717 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66717/consoleFull)**
for PR 15148 at commit
[`2c95e5c`](https://github.com/apache/spark/commit/
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/13675
Does this affect R code - could we add some R tests for this?
---
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 projec
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15072#discussion_r82725229
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -53,7 +53,15 @@ import org.apache.spark.util.Utils
private[sql] objec
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15424
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15408
Yea pooling can make sense, but we don't do it anywhere right now so it'd
make more sense to defer until we have a plan to do it more broadly.
---
If your project is set up for it, you can reply to t
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15426
cc @marmbrus
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15426
**[Test build #66721 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66721/consoleFull)**
for PR 15426 at commit
[`0cf7e72`](https://github.com/apache/spark/commit/0
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/15426
[SPARK-17864][SQL] Mark data type APIs as stable (not DeveloperApi)
## What changes were proposed in this pull request?
The data type API has not been changed since Spark 1.3.0, and is ready for
g
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15424
**[Test build #66720 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66720/consoleFull)**
for PR 15424 at commit
[`0ff26d0`](https://github.com/apache/spark/commit/0
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/15416#discussion_r82723882
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -298,10 +298,14 @@ class Analyzer(
case oth
Github user loneknightpy commented on the issue:
https://github.com/apache/spark/pull/15285
@tdas Addressed your comments
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/15408
Barring query to @rxin (regarding buffer pooling), I am fine with the
change - pretty neat, thanks @sitalkedia !
Would be good if more eyeballs look at it though given how fundamental it
is.
-
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15292
Ah, right. I just updated.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15377
**[Test build #66719 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66719/consoleFull)**
for PR 15377 at commit
[`df28bdd`](https://github.com/apache/spark/commit/d
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15292
Sorry, I did not explain it in details. In this PR, we had a bug fix. We
need a separate bullet in the PR description.
Previously, when attempting to make a database connection, we pass
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15421
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66702/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15421
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/15421
**[Test build #66702 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66702/consoleFull)**
for PR 15421 at commit
[`9e621eb`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15414
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66710/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15414
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/15414
**[Test build #66710 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66710/consoleFull)**
for PR 15414 at commit
[`6c61e73`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15285
**[Test build #66718 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66718/consoleFull)**
for PR 15285 at commit
[`e5676a6`](https://github.com/apache/spark/commit/e
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15398
Also cc @yhuai and @JoshRosen @mengxr Please check whether the changes here
can satisfy what you want. Thanks!
---
If your project is set up for it, you can reply to this email and have your
rep
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15425
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15272
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 Yunni commented on a diff in the pull request:
https://github.com/apache/spark/pull/15148#discussion_r82722577
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/LSH.scala ---
@@ -0,0 +1,343 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) unde
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15272
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66705/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15272
**[Test build #66705 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66705/consoleFull)**
for PR 15272 at commit
[`e9f9378`](https://github.com/apache/spark/commit/
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/15398#discussion_r82722525
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/StringUtils.scala
---
@@ -25,26 +25,25 @@ object StringUtils {
// repla
Github user seyfe commented on the issue:
https://github.com/apache/spark/pull/15371
Thanks @zsxwing.
Here is the PR for branch-2.0
https://github.com/apache/spark/pull/15425
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
GitHub user seyfe opened a pull request:
https://github.com/apache/spark/pull/15425
[SPARK-17816] [Core] [Branch-2.0] Fix ConcurrentModificationException issue
in BlockStatusesAccumulator
## What changes were proposed in this pull request?
Replaced `BlockStatusesAccumulator` wit
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15424#discussion_r82722351
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/catalog/Catalog.scala ---
@@ -270,9 +270,10 @@ abstract class Catalog {
* tied to any databases,
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15424
LGTM other than the two minor comments.
We also need a Python API for this, don't we?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
1 - 100 of 799 matches
Mail list logo