Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20907
One question I have is, how do users know if a model (e.g. KMeansModel
after this change) supports pmml & internal formats? I did a search on the
current docs, but didn't find any info.
---
--
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20907
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183318946
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/FileDataSourceV2.scala
---
@@ -0,0 +1,43 @@
+/*
+ * Licensed
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183318856
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -89,8 +91,13 @@ case class DataSource(
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r183318881
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -213,6 +215,25 @@ case class DataSourceA
Github user dedunumax commented on the issue:
https://github.com/apache/spark/pull/21120
Having sum as a basic statistic will make the API user-friendly. I'm
thinking about implementing other functions as well. Do you think it is not
worth to implement this?
---
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21018
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21018
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/2579/
Tes
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/20345
ping
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21018
**[Test build #89710 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89710/testReport)**
for PR 21018 at commit
[`9c9f9c2`](https://github.com/apache/spark/commit/9c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20980
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/2578/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20980
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20980
**[Test build #89709 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89709/testReport)**
for PR 20980 at commit
[`eaef6b3`](https://github.com/apache/spark/commit/ea
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20933
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/20980
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20933
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/2577/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20933
**[Test build #89708 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89708/testReport)**
for PR 20933 at commit
[`fd70a21`](https://github.com/apache/spark/commit/fd
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21072#discussion_r183311275
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -736,12 +736,22 @@ object EliminateSorts extends Rule
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21083
lgtm
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.a
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21083
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apac
Github user efimpoberezkin commented on the issue:
https://github.com/apache/spark/pull/21063
@jose-torres Hi Jose, could you review and tell me if we need more tests or
a single one will do? Also is it fine to assert status after every stream
action?
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21126
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/2576/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21126
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21126
**[Test build #89707 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89707/testReport)**
for PR 21126 at commit
[`d485db8`](https://github.com/apache/spark/commit/d4
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/21126
jenkins retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21083#discussion_r183299130
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -664,53 +662,52 @@ object InferFiltersFromConstraint
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/2
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/2575/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/2
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/2
**[Test build #89706 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89706/testReport)**
for PR 2 at commit
[`d624955`](https://github.com/apache/spark/commit/d6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21120
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21120
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89703/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21120
**[Test build #89703 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89703/testReport)**
for PR 21120 at commit
[`8c34fc9`](https://github.com/apache/spark/commit/8
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/2
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21100#discussion_r183296378
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -111,6 +111,14 @@ object TypeCoercion {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21100
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21100
**[Test build #89704 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89704/testReport)**
for PR 21100 at commit
[`0845739`](https://github.com/apache/spark/commit/08
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21083
**[Test build #89705 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89705/testReport)**
for PR 21083 at commit
[`787cddf`](https://github.com/apache/spark/commit/78
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21083#discussion_r183295844
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -664,53 +662,52 @@ object InferFiltersFromConstr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21083
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21083
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/2574/
Tes
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21083
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21100#discussion_r183295681
--- Diff: sql/core/src/test/resources/sql-tests/inputs/union.sql ---
@@ -35,6 +35,11 @@ FROM (SELECT col AS col
SELECT col
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21100#discussion_r183295566
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala
---
@@ -111,6 +111,14 @@ object TypeCoercion {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21100
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/21120
Why do you want to add this? Once we have mean, it's easy to compute sum.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21120
**[Test build #89703 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89703/testReport)**
for PR 21120 at commit
[`8c34fc9`](https://github.com/apache/spark/commit/8c
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/21120
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/19222
I think that one memory block in each iteration is more representative with
having possibility of megamorphism. This is because in the typicalusages in
Spark, a data structure is actually dominated by
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21100
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89696/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/2
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/2
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89695/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20940
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20980
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21100
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21126
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21100
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21083
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21126
**[Test build #89702 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89702/testReport)**
for PR 21126 at commit
[`d485db8`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21100
**[Test build #89700 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89700/testReport)**
for PR 21100 at commit
[`0845739`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20980
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89698/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21126
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89702/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20940
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89697/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21100
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89700/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/2
**[Test build #89695 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89695/testReport)**
for PR 2 at commit
[`d624955`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21083
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89701/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20980
**[Test build #89698 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89698/testReport)**
for PR 20980 at commit
[`eaef6b3`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21083
**[Test build #89701 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89701/testReport)**
for PR 21083 at commit
[`787cddf`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21100
**[Test build #89696 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89696/testReport)**
for PR 21100 at commit
[`19b5c6a`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20940
**[Test build #89697 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89697/testReport)**
for PR 20940 at commit
[`ae8a388`](https://github.com/apache/spark/commit/a
501 - 569 of 569 matches
Mail list logo