Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21070#discussion_r186602846
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedPlainValuesReader.java
---
@@ -63,115 +59,157 @@ public
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21263
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90350/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21263
**[Test build #90350 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90350/testReport)**
for PR 21263 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21263
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21129
**[Test build #4170 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4170/testReport)**
for PR 21129 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21264
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21264
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user yotingting opened a pull request:
https://github.com/apache/spark/pull/21264
Branch 2.2
## What changes were proposed in this pull request?
when i use yarn client mode, the spark task locked in the collect stage
countDf.collect().map(_.getLong(0)).mkString.toLong
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21209
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90343/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21209
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21209
**[Test build #90343 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90343/testReport)**
for PR 21209 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20929
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90345/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20929
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20929
**[Test build #90345 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90345/testReport)**
for PR 20929 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21257#discussion_r186600573
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InsertIntoHadoopFsRelationCommand.scala
---
@@ -207,9 +207,25 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21263
**[Test build #90350 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90350/testReport)**
for PR 21263 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21257#discussion_r186600352
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InsertIntoHadoopFsRelationCommand.scala
---
@@ -207,9 +207,25 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21263
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21259#discussion_r186600116
--- Diff: docs/sql-programming-guide.md ---
@@ -1812,6 +1812,8 @@ working with timestamps in `pandas_udf`s to get the
best performance, see
-
GitHub user caneGuy opened a pull request:
https://github.com/apache/spark/pull/21263
[SPARK-24084][ThriftServer] Add job group id for sql statement through
spark-sql
## What changes were proposed in this pull request?
Add job group id for spark-sql mode
## How
Github user zheh12 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21257#discussion_r186599760
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InsertIntoHadoopFsRelationCommand.scala
---
@@ -207,9 +207,25 @@ case class
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21118
> If we did that, then only ProjectExec and FilterExec would need to
support InternalRow, which they already do.
This partially brings #10511 back, and we need to plan project and filter
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90348/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21211
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21211
**[Test build #90348 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90348/testReport)**
for PR 21211 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21145#discussion_r186597255
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/ReadTask.java ---
@@ -22,20 +22,20 @@
import
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21259
Could you review this backward compatibility PR, @cloud-fan , @gatorsmile ,
@mridulm , and @HyukjinKwon ?
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21253
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90342/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21253
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21230
Hi @rdblue , thanks for your new approach! Like you said, the major problem
is about statistics. This is unfortunately a problem of Spark's CBO design: the
statistics should belong to physical
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21253
**[Test build #90342 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90342/testReport)**
for PR 21253 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21173
@gatorsmile ping
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21211
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3021/
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20929#discussion_r186593019
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/TypePlaceholder.scala ---
@@ -0,0 +1,23 @@
+/*
+ * Licensed to the Apache Software
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21199
**[Test build #90349 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90349/testReport)**
for PR 21199 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21211
**[Test build #90348 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90348/testReport)**
for PR 21211 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21199
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21199
add to whitelist
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/21092#discussion_r186591946
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/submit/KubernetesDriverBuilder.scala
---
@@ -29,18 +31,36 @@
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21211
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/21092#discussion_r186591608
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/submit/KubernetesDriverBuilder.scala
---
@@ -29,18 +31,36 @@
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/21092#discussion_r186591836
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/submit/KubernetesDriverBuilder.scala
---
@@ -29,18 +31,36 @@
Github user mccheah commented on a diff in the pull request:
https://github.com/apache/spark/pull/21092#discussion_r186591534
--- Diff:
resource-managers/kubernetes/core/src/main/scala/org/apache/spark/deploy/k8s/features/bindings/PythonDriverFeatureStep.scala
---
@@ -0,0 +1,72
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20929#discussion_r186589216
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/TypePlaceholder.scala ---
@@ -0,0 +1,23 @@
+/*
+ * Licensed to the Apache Software
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21259
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21259
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90341/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21259
**[Test build #90341 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90341/testReport)**
for PR 21259 at commit
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/21145#discussion_r186587220
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/reader/ReadTask.java ---
@@ -22,20 +22,20 @@
import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21254
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3020/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21254
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21254
**[Test build #90347 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90347/testReport)**
for PR 21254 at commit
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21118
@cloud-fan: This PR is also related to #21262 because that PR updates the
conversion from logical to physical plan and handles projections and filtering.
We could modify that strategy to always
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21254
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21262
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3019/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21262
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21230
@cloud-fan, I opened #21262 that is similar to this, but does pushdown when
converting to a physical plan. You might like that as an alternative because it
cleans up `DataSourceV2Relation` quite a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21262
**[Test build #90346 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90346/testReport)**
for PR 21262 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21250
Thanks, @cloud-fan, @yanboliang, @felixcheung and @viirya.
---
-
To unsubscribe, e-mail:
GitHub user rdblue opened a pull request:
https://github.com/apache/spark/pull/21262
[SPARK-24172][SQL]: Push projection and filters once when converting to
physical plan.
## What changes were proposed in this pull request?
This removes `PruneFileSourcePartitions` and
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21249
Thanks, @cloud-fan, @yanboliang, @felixcheung and @viirya.
---
-
To unsubscribe, e-mail:
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20929#discussion_r186584474
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/types/TypePlaceholder.scala ---
@@ -0,0 +1,23 @@
+/*
+ * Licensed to the Apache Software
Github user HyukjinKwon closed the pull request at:
https://github.com/apache/spark/pull/21250
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user bersprockets commented on the issue:
https://github.com/apache/spark/pull/21144
Thanks much!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21239
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90340/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21239
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21239
**[Test build #90340 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90340/testReport)**
for PR 21239 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20929
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3018/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20929
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20929
**[Test build #90345 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90345/testReport)**
for PR 20929 at commit
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20929#discussion_r186581349
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -887,6 +887,14 @@ object SQLConf {
.booleanConf
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/20929#discussion_r186581071
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -887,6 +887,14 @@ object SQLConf {
.booleanConf
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/20929
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21106#discussion_r186580919
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CodegenObjectFactory.scala
---
@@ -0,0 +1,80 @@
+/*
+ * Licensed to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21195
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21195
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90344/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21195
**[Test build #90344 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90344/testReport)**
for PR 21195 at commit
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/21129#discussion_r186572477
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -497,6 +498,9 @@ private[ml] trait GBTParams extends TreeEnsembleParams
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/21129#discussion_r186572374
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/GBTClassifierSuite.scala
---
@@ -365,6 +366,50 @@ class GBTClassifierSuite extends
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/21129#discussion_r186573136
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/GBTClassifier.scala ---
@@ -146,20 +146,40 @@ class GBTClassifier @Since("1.4.0") (
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/21129#discussion_r186569928
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/GBTClassifierSuite.scala
---
@@ -365,6 +366,50 @@ class GBTClassifierSuite extends
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/21129#discussion_r186572853
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/GBTClassifier.scala ---
@@ -146,20 +146,40 @@ class GBTClassifier @Since("1.4.0") (
Github user bersprockets commented on the issue:
https://github.com/apache/spark/pull/21073
@ueshin Hopefully I have addressed all of your review comments.
Also, I have a question about what it means to dedup across maps when Spark
allows duplicates in maps
Github user bersprockets commented on a diff in the pull request:
https://github.com/apache/spark/pull/21073#discussion_r186570491
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -116,6 +117,169 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21258
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21258
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90338/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21258
**[Test build #90338 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90338/testReport)**
for PR 21258 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21073
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21073
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90336/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21129
**[Test build #4170 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4170/testReport)**
for PR 21129 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21073
**[Test build #90336 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90336/testReport)**
for PR 21073 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21251#discussion_r186567342
--- Diff:
core/src/test/scala/org/apache/spark/deploy/SparkSubmitUtilsSuite.scala ---
@@ -256,4 +256,20 @@ class SparkSubmitUtilsSuite extends SparkFunSuite
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21251#discussion_r186567456
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -1204,7 +1205,36 @@ private[spark] object SparkSubmitUtils {
/**
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21251#discussion_r186566903
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -1204,7 +1205,36 @@ private[spark] object SparkSubmitUtils {
/**
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/21097#discussion_r186565756
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GBTRegressor.scala ---
@@ -269,6 +269,20 @@ class GBTRegressionModel private[ml](
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21195
**[Test build #90344 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90344/testReport)**
for PR 21195 at commit
Github user ludatabricks commented on a diff in the pull request:
https://github.com/apache/spark/pull/21195#discussion_r186566521
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/clustering/LDASuite.scala ---
@@ -323,4 +324,21 @@ class LDASuite extends SparkFunSuite with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21209
**[Test build #90343 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90343/testReport)**
for PR 21209 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21209
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20261
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
101 - 200 of 658 matches
Mail list logo
