Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20126
Hm, I see. Will open a followup PR.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional command
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20126
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85560/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20126
**[Test build #85560 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85560/testReport)**
for PR 20126 at commit
[`2a9dff4`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20126
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/20126
**[Test build #85560 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85560/testReport)**
for PR 20126 at commit
[`2a9dff4`](https://github.com/apache/spark/commit/2a
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20069#discussion_r159141412
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -851,7 +851,7 @@ object PushDownPredicate extends Rule[
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20126
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/20126
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85559/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20126
**[Test build #85559 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85559/testReport)**
for PR 20126 at commit
[`23cc79b`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20126
**[Test build #85559 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85559/testReport)**
for PR 20126 at commit
[`23cc79b`](https://github.com/apache/spark/commit/23
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20125
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/20125
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/8/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20125
**[Test build #8 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/8/testReport)**
for PR 20125 at commit
[`5cae64b`](https://github.com/apache/spark/commit/5
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20125
I actually think
https://github.com/apache/spark/pull/20125#issuecomment-354604768 are good
points and I was hesitant about it. Although IMHO I think it's fine but let me
cc @hvanhovell and @rx
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20125
> Btw, is this any difference than using string? Like:
Nope, they will be the same but I was thinking this is a simplest fix.
---
--
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20125
Yup, I was thinking of SparkSQL only feature.
For more details, the original intention was to support multiple values for
`nullValue` but I realised such option support can be generallis
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20125
Is this a special feature for SparkSQL only? Seems Hive doesn't have such
support.
---
-
To unsubscribe, e-mail: reviews-unsubscr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20126
**[Test build #85558 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85558/testReport)**
for PR 20126 at commit
[`85639dd`](https://github.com/apache/spark/commit/85
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20126
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/20126
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85556/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20126
**[Test build #85556 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85556/testReport)**
for PR 20126 at commit
[`ada1c4c`](https://github.com/apache/spark/commit/a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20127
**[Test build #85557 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85557/testReport)**
for PR 20127 at commit
[`f158a95`](https://github.com/apache/spark/commit/f1
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20127
cc @cloud-fan @jiangxb1987
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/20127
[SPARK-22932] [SQL] Refactor AnalysisContext
## What changes were proposed in this pull request?
Add a `reset` function to ensure the state in `AnalysisContext ` is
per-query.
## H
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20126
**[Test build #85556 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85556/testReport)**
for PR 20126 at commit
[`ada1c4c`](https://github.com/apache/spark/commit/ad
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/20126
[DO-NOT-MERGE] Investigate if changes in flume.py actually triggeres
related tests
## What changes were proposed in this pull request?
Do not merge this.
Seems the changes in
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19991
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/19991
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19715
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/19715
Merged to master. If there are any further small comments / clean ups we
can do that during QA for 2.3
Thanks @huaxingao and all others for review!
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20114
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85554/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20114
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/20114
**[Test build #85554 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85554/testReport)**
for PR 20114 at commit
[`281ffdc`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20125
**[Test build #8 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/8/testReport)**
for PR 20125 at commit
[`5cae64b`](https://github.com/apache/spark/commit/5c
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20125
cc @gatorsmile could you take a look please?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additiona
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/20125
[SPARK-17967][SQL] Support for array as an option in SQL parser
## What changes were proposed in this pull request?
This PR targets to add the ability for dealing with an array (JSON ar
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20076#discussion_r159136922
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/CompressionCodecPrecedenceSuite.scala
---
@@ -0,0 +1,60 @@
+/*
+ * Licensed to the Apache Sof
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20124
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/20124
**[Test build #85553 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85553/testReport)**
for PR 20124 at commit
[`53521ca`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20124
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85553/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20114
**[Test build #85554 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85554/testReport)**
for PR 20114 at commit
[`281ffdc`](https://github.com/apache/spark/commit/28
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/20114
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/20124
This basically works by splitting the array of ParamMaps into two. One
that has params that can be optimized by the estimator, and one that can be
parallelized over. These are then grouped tog
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20124
**[Test build #85553 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85553/testReport)**
for PR 20124 at commit
[`53521ca`](https://github.com/apache/spark/commit/53
GitHub user BryanCutler opened a pull request:
https://github.com/apache/spark/pull/20124
[WIP][SPARK-22126][ML] Fix model-specific optimization support for ML
tuning.
## What changes were proposed in this pull request?
Support model-specific optimizations for CrossValidato
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20114
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/85552/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20114
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/20114
**[Test build #85552 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85552/testReport)**
for PR 20114 at commit
[`281ffdc`](https://github.com/apache/spark/commit/2
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20072#discussion_r159135028
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -261,6 +261,17 @@ object SQLConf {
.booleanConf
.cr
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20072#discussion_r159134987
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -261,6 +261,17 @@ object SQLConf {
.booleanConf
.cr
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20072#discussion_r159135036
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/HadoopFsRelation.scala
---
@@ -60,6 +60,8 @@ case class HadoopFsRelation(
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/20072#discussion_r159135272
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/HadoopFsRelation.scala
---
@@ -82,7 +84,15 @@ case class HadoopFsRelation(
101 - 152 of 152 matches
Mail list logo