Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21192
eh I actually think separated makes it much simpler to look at, compared
with an array. Why complicate the API and require users to understand how to
specify an array (in all languages)?
One
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/20894#discussion_r188841673
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVDataSource.scala
---
@@ -206,24 +280,33 @@ object
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/20894#discussion_r188841632
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVDataSource.scala
---
@@ -202,28 +263,33 @@ object
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21350
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/21028
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/3282/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21350
**[Test build #90709 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90709/testReport)**
for PR 21350 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21350
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90709/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21028
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
@kevinyu98 Thanks for you feedback, i will test once all my scenario with
your fix, but here one more point i need to mention is like my fix has also has
impact in the logic of local path
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21252
can you change the PR title? otherwise LGTM
---
-
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/21028#discussion_r188840260
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -18,15 +18,53 @@ package
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21028
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/21028
**[Test build #90711 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90711/testReport)**
for PR 21028 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21028
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90711/
Test FAILed.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21028
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user efimpoberezkin commented on the issue:
https://github.com/apache/spark/pull/20936
@tdas can't start tests
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21327
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/21327
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/3281/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21348
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90701/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21348
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/21028
**[Test build #90711 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90711/testReport)**
for PR 21028 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21327
**[Test build #90710 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90710/testReport)**
for PR 21327 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21348
**[Test build #90701 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90701/testReport)**
for PR 21348 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21028
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21327
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/21345
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90702/
Test PASSed.
---
Github user efimpoberezkin commented on the issue:
https://github.com/apache/spark/pull/20936
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/21345
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/21345
**[Test build #90702 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90702/testReport)**
for PR 21345 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21086
@ghoto Sure. Will backport it tonight.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21350
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/21350
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/3280/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21350
**[Test build #90709 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90709/testReport)**
for PR 21350 at commit
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/21350
[SPARK-24303][PYTHON] Update cloudpickle to v0.4.4
## What changes were proposed in this pull request?
cloudpickle 0.4.4 is release -
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21350
cc @ueshin and @BryanCutler
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21208
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/21208
Thanks! merging to master.
---
-
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/21349
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/21349
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/3279/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21349
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/21349
Test FAILed.
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/3278/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21349
**[Test build #90708 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90708/testReport)**
for PR 21349 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21267
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21349
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90707/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21349
**[Test build #90707 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90707/testReport)**
for PR 21349 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21349
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21267
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21349
**[Test build #90707 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90707/testReport)**
for PR 21349 at commit
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/21349
[MINOR][PROJECT-INFRA] Check if 'original_head' variable is defined in
clean_up at merge script
## What changes were proposed in this pull request?
This PR proposes to check if global
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21268
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90698/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21268
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/21268
**[Test build #90698 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90698/testReport)**
for PR 21268 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21337
**[Test build #90706 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90706/testReport)**
for PR 21337 at commit
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/21337
@tdas
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21337#discussion_r188831887
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/continuous/shuffle/ContinuousShuffleReadSuite.scala
---
@@ -0,0 +1,122 @@
+/*
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21267
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90704/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21267
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20146
ping @dbtsai @jkbradley
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21267
**[Test build #90704 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90704/testReport)**
for PR 21267 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21193
Thanks @maropu @kiszk @mgaido91
@mgaido91 I overlooked it. Just replied you.
---
-
To unsubscribe, e-mail:
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21193#discussion_r188830465
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/javaCode.scala
---
@@ -114,6 +114,113 @@ object JavaCode {
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21337
**[Test build #90705 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90705/testReport)**
for PR 21337 at commit
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21337#discussion_r188829894
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/shuffle/ContinuousShuffleReadRDD.scala
---
@@ -0,0 +1,64 @@
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21337#discussion_r188829815
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/shuffle/ContinuousShuffleReadRDD.scala
---
@@ -0,0 +1,64 @@
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21337#discussion_r188829780
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/shuffle/UnsafeRowReceiver.scala
---
@@ -0,0 +1,56 @@
+/*
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21299#discussion_r188828684
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SQLExecution.scala ---
@@ -90,13 +92,42 @@ object SQLExecution {
* thread from
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21267
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/21267
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/3277/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21267
**[Test build #90704 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90704/testReport)**
for PR 21267 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21267
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21346
All good questions and stuff I had wondered about too -- I should actually
be sure to comment on these on the jira as well:
> I recall that the problem with large shuffle blocks was that the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21114
**[Test build #90703 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90703/testReport)**
for PR 21114 at commit
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/21114
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/21347#discussion_r188820444
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/util/Instrumentation.scala ---
@@ -125,6 +125,19 @@ private[spark] class Instrumentation[E <:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21345
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/21345
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/3276/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21345
**[Test build #90702 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90702/testReport)**
for PR 21345 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21348
**[Test build #90701 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90701/testReport)**
for PR 21348 at commit
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/21090#discussion_r188813735
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/clustering/PowerIterationClustering.scala
---
@@ -0,0 +1,256 @@
+/*
+ * Licensed to the
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/21342
I'm also in favor of delaying for a couple of days for more detailed review
because historically I think these types of changes have been high risk. The
risk calculus might be a bit different if
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/20973#discussion_r188813405
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/PrefixSpan.scala ---
@@ -0,0 +1,96 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21183
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/20973#discussion_r188813297
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/PrefixSpan.scala ---
@@ -0,0 +1,96 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/21183
Thanks for checking this manually. Since the test sometimes fails, then
let's leave it.
LGTM
Merging with master
Thanks @ludatabricks and @mengxr !
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21348
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/21348
**[Test build #90700 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90700/testReport)**
for PR 21348 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21348
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90700/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21348
**[Test build #90700 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90700/testReport)**
for PR 21348 at commit
Github user bdrillard closed the pull request at:
https://github.com/apache/spark/pull/20085
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user bdrillard commented on the issue:
https://github.com/apache/spark/pull/20085
Closing this PR in favor of #21348.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user DylanGuedes commented on the issue:
https://github.com/apache/spark/pull/21045
Great! Also I finally discovered the unit-tests log file that logs the
generated java code :D
Since now it works I'll remove the WIP tag and focus on the other
suggestions (such as new
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20894
Right, will proceed reviewing this.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21348
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/21348
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90699/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21348
**[Test build #90699 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90699/testReport)**
for PR 21348 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21348
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/21348
**[Test build #90699 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90699/testReport)**
for PR 21348 at commit
Github user bdrillard commented on a diff in the pull request:
https://github.com/apache/spark/pull/21348#discussion_r188810286
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/objects/objects.scala
---
@@ -408,16 +439,19 @@ object NewInstance {
GitHub user bdrillard opened a pull request:
https://github.com/apache/spark/pull/21348
[SPARK-22739][Catalyst Additional Expression Support for Objects
## What changes were proposed in this pull request?
This PR is a working followup to the expression work begun in #20085.
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21342
Could we hold this PR? This change sounds risky. `OutOfMemoryError` is not
issued by Spark. If we eat it, it could cause serious outcomes. cc @zsxwing
@JoshRosen
---
1 - 100 of 426 matches
Mail list logo
