Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16785
**[Test build #72303 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72303/testReport)**
for PR 16785 at commit
[`b4e514a`](https://github.com/apache/spark/commit/b4
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16775
For the issue reported on mailing list, I found the root cause makes
significant difference between 1.6 and current branch. The fix is at #16785.
However, I think this patch is still useful.
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/16785
[SPARK-19443][SQL] The function to generate constraints takes too long when
the query plan grows continuously
## What changes were proposed in this pull request?
This issue is originally re
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99286995
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -117,6 +134,34 @@ class DatasetSuite extends QueryTest with
SharedSQLCon
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99286957
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -17,24 +17,41 @@
package org.apache.spark.sql
-impor
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99286805
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -117,6 +134,34 @@ class DatasetSuite extends QueryTest with
SharedSQLCon
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16779
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99286475
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -117,6 +134,34 @@ class DatasetSuite extends QueryTest with
SharedSQLCon
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/16779
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 wish
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99286218
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala
---
@@ -497,7 +496,9 @@ case class UnionExec(children: Seq[
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99286066
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala
---
@@ -19,9 +19,8 @@ package org.apache.spark.sql.executio
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16784
**[Test build #72302 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72302/testReport)**
for PR 16784 at commit
[`fc1f7d1`](https://github.com/apache/spark/commit/fc
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99285902
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -823,6 +825,17 @@ case class Reparti
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99285925
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -823,6 +825,17 @@ case class Reparti
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99285876
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -823,6 +825,17 @@ case class Reparti
GitHub user wangmiao1981 opened a pull request:
https://github.com/apache/spark/pull/16784
[SPARK-19382][ML]:Test sparse vectors in LinearSVCSuite
## What changes were proposed in this pull request?
Add unit tests for testing SparseVector.
We can't add mixed DenseVe
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99285447
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -17,7 +17,9 @@
package or
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99284849
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala
---
@@ -497,7 +496,9 @@ case class UnionExec(children: Seq[
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16766#discussion_r99284809
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -2437,9 +2435,12 @@ class Dataset[T] private[sql](
* @group typedrel
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16765
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 fe
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16777#discussion_r99283834
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -101,24 +101,13 @@ object TypeCoercion {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16138
**[Test build #72301 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72301/testReport)**
for PR 16138 at commit
[`a2d0221`](https://github.com/apache/spark/commit/a2
Github user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/16138#discussion_r99283708
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -1047,6 +1048,64 @@ case class ToDate(chi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16779
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/16779
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72297/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16779
**[Test build #72297 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72297/testReport)**
for PR 16779 at commit
[`a9bc3f4`](https://github.com/apache/spark/commit/a
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16740
@seth @imatiach-msft Let me know if there is any other changes needed.
Thanks much for your review!
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16765
@gatorsmile Updated. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16740
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/16740
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72299/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16740
**[Test build #72299 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72299/testReport)**
for PR 16740 at commit
[`b57af08`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16783
**[Test build #72300 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72300/testReport)**
for PR 16783 at commit
[`127a114`](https://github.com/apache/spark/commit/12
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/16783
[SPARK-19441] [SQL] Remove IN type coercion from PromoteStrings
### What changes were proposed in this pull request?
The removed codes are not reachable, because `InConversion` already resolv
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16740
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72298/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16740
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/16740
**[Test build #72298 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72298/testReport)**
for PR 16740 at commit
[`931f7ec`](https://github.com/apache/spark/commit/9
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14702
I will try to review it in the next few days. Thanks for working on 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 pr
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16765
Could you update the PR description and title? This PR fixes three
scenarios:
- `except`on two Datasets with UDT
- `intersect` on two Datasets with UDT
- `Join` with the join conditi
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r99279786
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/DecisionTreeClassifierSuite.scala
---
@@ -351,6 +370,36 @@ class DecisionTreeClassif
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r99279066
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/tree/ImpuritySuite.scala ---
@@ -18,23 +18,62 @@
package org.apache.spark.mllib.tree
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r99278975
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/util/MLTestingUtils.scala ---
@@ -281,10 +283,26 @@ object MLTestingUtils extends SparkFunSuite {
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r99278910
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/tree/impl/TreeTests.scala ---
@@ -124,8 +129,8 @@ private[ml] object TreeTests extends SparkFunSuite
Github user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/16138#discussion_r99278789
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -1177,6 +1177,9 @@ test_that("column functions", {
c17 <- cov(c, c1) + cov("c", "c1") + co
Github user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/16138#discussion_r99278746
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -1047,6 +1048,64 @@ case class ToDate(chi
Github user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/16138#discussion_r99278738
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -1047,6 +1048,64 @@ case class ToDate(chi
Github user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/16138#discussion_r99278624
--- Diff: R/pkg/R/functions.R ---
@@ -1746,7 +1750,7 @@ setMethod("toRadians",
#' to_date(df$c)
#' to_date(df$c, '-MM-dd')
#' }
-#' @
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r99278201
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/DecisionTreeClassifierSuite.scala
---
@@ -58,6 +62,20 @@ class DecisionTreeClassifie
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r99278110
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/tree/impurity/Variance.scala ---
@@ -70,17 +70,24 @@ object Variance extends Impurity {
* N
Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16740#discussion_r99277921
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/GeneralizedLinearRegressionSuite.scala
---
@@ -743,6 +744,48 @@ class GeneralizedLinearRe
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r99277799
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/tree/impurity/Gini.scala ---
@@ -80,23 +80,29 @@ object Gini extends Impurity {
* @param nu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16740
**[Test build #72299 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72299/testReport)**
for PR 16740 at commit
[`b57af08`](https://github.com/apache/spark/commit/b5
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16740#discussion_r99276472
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/GeneralizedLinearRegressionSuite.scala
---
@@ -743,6 +744,48 @@ class GeneralizedLinearR
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16740
**[Test build #72298 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72298/testReport)**
for PR 16740 at commit
[`931f7ec`](https://github.com/apache/spark/commit/93
Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16740#discussion_r99276315
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -335,6 +335,11 @@ class GeneralizedLinearRegress
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r99275923
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/tree/impurity/Entropy.scala ---
@@ -83,23 +83,29 @@ object Entropy extends Impurity {
* @pa
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r99275700
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/tree/impurity/Impurity.scala ---
@@ -79,7 +79,12 @@ private[spark] abstract class ImpurityAggrega
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16722#discussion_r99274459
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/tree/impurity/Entropy.scala ---
@@ -83,23 +83,29 @@ object Entropy extends Impurity {
* @pa
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/16733#discussion_r99273860
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/jdbc/OracleDialect.scala ---
@@ -29,7 +29,12 @@ private case object OracleDialect extends JdbcDialect {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16733#discussion_r99273809
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/jdbc/OracleDialect.scala ---
@@ -29,7 +29,12 @@ private case object OracleDialect extends JdbcDiale
Github user imatiach-msft commented on a diff in the pull request:
https://github.com/apache/spark/pull/16740#discussion_r99273642
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -335,6 +335,11 @@ class GeneralizedLinearRegres
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16765
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72295/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16765
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/16765
**[Test build #72295 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72295/testReport)**
for PR 16765 at commit
[`ac3c3bf`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16779
**[Test build #72297 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72297/testReport)**
for PR 16779 at commit
[`a9bc3f4`](https://github.com/apache/spark/commit/a9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16740
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72296/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16740
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 zsxwing commented on the issue:
https://github.com/apache/spark/pull/16779
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 the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16740
**[Test build #72296 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72296/testReport)**
for PR 16740 at commit
[`3a0a2af`](https://github.com/apache/spark/commit/3
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16739
: ) This might be caused by the optimizer rule `CollapseRepartition`. Can
you output the plan by `explain(true)`?
---
If your project is set up for it, you can reply to this email and have your
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/15435
@sethah
If I merge the MulticlassLogisticRegressionSummary into
LogisticRegressionSummary,
then, according to the hierarchy currently designed, it became:
class LogisticRe
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/16740#discussion_r99269075
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/GeneralizedLinearRegressionSuite.scala
---
@@ -743,6 +743,55 @@ class GeneralizedLinearRegressi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16740
**[Test build #72296 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72296/testReport)**
for PR 16740 at commit
[`3a0a2af`](https://github.com/apache/spark/commit/3a
Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16740#discussion_r99269006
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/GeneralizedLinearRegressionSuite.scala
---
@@ -743,6 +743,55 @@ class GeneralizedLinearRe
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/16779
@zsxwing
Thanks a lot for reviewing this. Not sure why the test doesn't start
automatically.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/16740#discussion_r99268773
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/GeneralizedLinearRegressionSuite.scala
---
@@ -743,6 +743,55 @@ class GeneralizedLinearRegressi
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/15435
sethah
About this issue:
Why is there a one-to-one overlap between MulticlassClassificationSummary
and LogisticRegressionSummary, and MulticlassLogisticRegressionSummary inherits
Github user actuaryzhang commented on the issue:
https://github.com/apache/spark/pull/16740
@srowen would you please take a look and merge this if all is good? Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user jinxing64 closed the pull request at:
https://github.com/apache/spark/pull/16780
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/16780
Thanks a lot for looking into this~ @zsxwing
You are right. My understanding about this is incorrect.
`CoarseGrainedSchedulerBackend: DriverEndpoint` is a `ThreadSafeRpcEndpoint`,
thus concur
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15264
Yeap, I will try to get that back after finishing up few issues I am
currently working on. I just realised that it'd take a bit of time for me to
proceed (as I noticed we need a more careful tou
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/16607#discussion_r99263532
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Word2Vec.scala
---
@@ -302,16 +302,36 @@ class Word2VecModel private[ml] (
@Since("1.6.0")
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/16607#discussion_r99263525
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Word2Vec.scala
---
@@ -320,14 +340,29 @@ object Word2VecModel extends
MLReadable[Word2VecModel]
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/16607#discussion_r99259617
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Word2Vec.scala
---
@@ -18,10 +18,9 @@
package org.apache.spark.ml.feature
import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16686
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72294/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16686
**[Test build #72294 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72294/testReport)**
for PR 16686 at commit
[`5b48fc6`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16686
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 sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/16699#discussion_r99263111
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/regression/GeneralizedLinearRegressionSuite.scala
---
@@ -743,6 +743,84 @@ class GeneralizedLinearRegressi
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14702
can anyone please review this PR ?
---
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
enab
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/16740
Ok, yeah, let's go with this fix now then - seems both R and statsmodels
fit to compute the null model. Thanks for following up on that!
---
If your project is set up for it, you can reply to this e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16765
**[Test build #72295 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72295/testReport)**
for PR 16765 at commit
[`ac3c3bf`](https://github.com/apache/spark/commit/ac
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/16733#discussion_r99261637
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/jdbc/OracleDialect.scala ---
@@ -29,7 +29,12 @@ private case object OracleDialect extends JdbcDialect {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16686
**[Test build #72294 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72294/testReport)**
for PR 16686 at commit
[`5b48fc6`](https://github.com/apache/spark/commit/5b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16607
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/16607
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72293/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16607
**[Test build #72293 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72293/testReport)**
for PR 16607 at commit
[`9b5e928`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16607
**[Test build #72293 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72293/testReport)**
for PR 16607 at commit
[`9b5e928`](https://github.com/apache/spark/commit/9b
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16607
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 t
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16607
Sorry for the delay; will take a look now!
---
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 featur
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14412
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/14412
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72291/
Test PASSed.
---
1 - 100 of 304 matches
Mail list logo
