Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21201
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21040
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/21202
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/21204
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/21195
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/21200
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/21199
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/21194
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/21194
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/21202
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/21199
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21204
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/21198#discussion_r185182067
--- Diff: python/run-tests.py ---
@@ -77,13 +79,33 @@ def run_individual_python_test(test_name,
pyspark_python):
'PYSPARK_PYTHON': which(py
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/21198#discussion_r185182113
--- Diff: python/run-tests.py ---
@@ -77,13 +79,33 @@ def run_individual_python_test(test_name,
pyspark_python):
'PYSPARK_PYTHON': which(py
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21202
@foxish
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21202
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21201
Jenkins, ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21200
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21200
Jenkins, add to whitelist
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21203
> perhaps better to match python's version_info ?
This for now is more close to `VersionUtils.majorMinorVersion`. To match
python's version_info, means we need to have 5 components?
---
--
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21136
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
GitHub user XD-DENG opened a pull request:
https://github.com/apache/spark/pull/21205
[Minor] To add a missing full-stop.
## What changes were proposed in this pull request?
In the document [Tuning Spark -> Determining Memory
Consumption](https://spark.apache.org/docs/lates
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21054#discussion_r185171846
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/regression.scala
---
@@ -0,0 +1,193 @@
+/*
+ * Licensed to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21205
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21054#discussion_r185170928
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Average.scala
---
@@ -23,23 +23,13 @@ import org.apache.spark.sql.
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21054#discussion_r185170552
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/Average.scala
---
@@ -62,14 +52,6 @@ case class Average(child: Exp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21205
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user ala commented on the issue:
https://github.com/apache/spark/pull/21206
@hvanhovell
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.a
GitHub user ala opened a pull request:
https://github.com/apache/spark/pull/21206
[SPARK-24133][SQL] Check for integer overflows when resizing
WritableColumnVectors
## What changes were proposed in this pull request?
`ColumnVector`s store string data in one big byte array.
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r185196808
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -191,28 +161,202 @@ case class SortArray(b
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r185197041
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -191,28 +161,202 @@ case class SortArray(b
Github user ala commented on the issue:
https://github.com/apache/spark/pull/21206
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21192#discussion_r185197844
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala
---
@@ -120,8 +120,26 @@ private[sql] class JSONOptions(
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/spark/pull/21200#discussion_r185198458
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/ContinuousQueuedDataReader.scala
---
@@ -0,0 +1,90 @@
+/*
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/spark/pull/21200#discussion_r185187424
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/ContinuousDataSourceRDD.scala
---
@@ -0,0 +1,153 @@
+/*
+
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/spark/pull/21200#discussion_r185194384
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/ContinuousDataSourceRDD.scala
---
@@ -0,0 +1,153 @@
+/*
+
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/spark/pull/21200#discussion_r185201032
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/WriteToContinuousDataSourceExec.scala
---
@@ -46,28 +46,34 @@ cas
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/spark/pull/21200#discussion_r18524
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/EpochPollRunnable.scala
---
@@ -0,0 +1,61 @@
+/*
+ * Lice
Github user mswit-databricks commented on the issue:
https://github.com/apache/spark/pull/21070
@rdblue Do you see any risk of additional overhead coming from the extra
stats? For example, if the data contains very long strings, performing
comparison on them to generate stats will be
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/21123#discussion_r185215070
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -213,6 +215,26 @@ case class DataSourceA
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21158
the original change was merged into 2.3.0, see
https://issues.apache.org/jira/browse/SPARK-22479. We are keeping the
behavior of that fix and that is to not show the credentials of the jdbc
con
Github user advancedxy commented on a diff in the pull request:
https://github.com/apache/spark/pull/21165#discussion_r185226136
--- Diff: core/src/main/scala/org/apache/spark/TaskEndReason.scala ---
@@ -212,9 +212,15 @@ case object TaskResultLost extends TaskFailedReason {
*
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21158
But the main thing is that the change modified the previous behavior which
was *not* to redact these things in the previous places where redaction
occurred. So, in a way, this change is fixing a regr
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21198#discussion_r185242337
--- Diff: python/run-tests.py ---
@@ -77,13 +79,33 @@ def run_individual_python_test(test_name,
pyspark_python):
'PYSPARK_PYTHON': which(pyspark
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21198#discussion_r185242826
--- Diff: python/run-tests.py ---
@@ -77,13 +79,33 @@ def run_individual_python_test(test_name,
pyspark_python):
'PYSPARK_PYTHON': which(pyspark
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21185
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21193
@hvanhovell Thanks for your comment!
I tried to add new JavaCode abstraction `Block` which holds the code to
generate. I hope this approach can be less side effect. Please comment if you
hav
Github user imatiach-msft commented on the issue:
https://github.com/apache/spark/pull/17086
jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21014
Merging to master / 2.3 / 2.2.
---
-
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/21014
---
-
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/18390
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21185
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21185
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21185
**[Test build #89978 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89978/testReport)**
for PR 21185 at commit
[`49783c5`](https://github.com/apache/spark/commit/49
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/21194#discussion_r185252544
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/sources/RateStreamProvider.scala
---
@@ -101,25 +101,10 @@ object RateStrea
Github user xuanyuanking commented on a diff in the pull request:
https://github.com/apache/spark/pull/21194#discussion_r185252360
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/sources/RateStreamProvider.scala
---
@@ -101,25 +101,10 @@ object RateStrea
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21206
cc @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21206
test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21206
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21206
**[Test build #89979 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89979/testReport)**
for PR 21206 at commit
[`17e2d02`](https://github.com/apache/spark/commit/17
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21206
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/21206
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/2751/
Tes
Github user Tagar commented on the issue:
https://github.com/apache/spark/pull/21187
@maryannxue, yep, "IN ANY" is what I meant. I missed it was already in the
description - thanks for clarifying this.
---
-
To unsu
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21185
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/2752/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21185
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/21206
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/2753/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21206
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21187
In-any-subquery in Pivot can be implemented like what we did in the other
parts, but let us first leave it as the future item. @maryannxue Maybe you can
create a JIRA.
The Oracle-compat
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21158
Also note that Tom's comment above is not 100% accurate - this change
*does* redact the JDBC URL on the SQL side, for reasons I described in my
comments on the bug. That's the whole reason why the SQ
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21198
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21187#discussion_r185258526
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -805,6 +810,7 @@ RIGHT: 'RIGHT';
FULL: 'FULL';
NA
Github user arunmahadevan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21200#discussion_r185259515
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/EpochPollRunnable.scala
---
@@ -0,0 +1,61 @@
+/*
+ * Li
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21198
**[Test build #89980 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89980/testReport)**
for PR 21198 at commit
[`77e8ff5`](https://github.com/apache/spark/commit/77
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21187#discussion_r185260075
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -691,7 +691,9 @@ case class Pivot(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21187#discussion_r185259275
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -691,7 +691,9 @@ case class Pivot(
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/21195
add to whitelist
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user squito commented on the issue:
https://github.com/apache/spark/pull/21185
I don't understand the comments about "leaked threads". the executor
thread pool is allowed to create threads whenever it wants. You can play
around with this example if you like:
https://gist.git
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/21153#discussion_r185262870
--- Diff: python/pyspark/util.py ---
@@ -61,6 +62,26 @@ def _get_argspec(f):
return argspec
+def majorMinorVersion(version):
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21195
**[Test build #89981 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89981/testReport)**
for PR 21195 at commit
[`45e6e96`](https://github.com/apache/spark/commit/45
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/20973
LGTM pending jenkins tests
---
-
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/20973
**[Test build #4162 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4162/testReport)**
for PR 20973 at commit
[`76d4119`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21204
**[Test build #4163 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4163/testReport)**
for PR 21204 at commit
[`7b75ed6`](https://github.com/apache/spark/commit/7
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21200#discussion_r185264079
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/EpochPollRunnable.scala
---
@@ -0,0 +1,61 @@
+/*
+ * Lice
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/20319
@smurakozi Do you have time to update this? I did a full review, though
it now has a small merge conflict. Thanks!
---
-
To
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21198
**[Test build #89980 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89980/testReport)**
for PR 21198 at commit
[`77e8ff5`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21198
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/21198
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89980/
Test FAILed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21187#discussion_r185264657
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala
---
@@ -691,7 +691,9 @@ case class Pivot(
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21203
Retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21197
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21196
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21200#discussion_r185265321
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/ContinuousDataSourceRDD.scala
---
@@ -0,0 +1,153 @@
+/*
+
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21197
**[Test build #89983 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89983/testReport)**
for PR 21197 at commit
[`c1c6377`](https://github.com/apache/spark/commit/c1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21203
**[Test build #89982 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89982/testReport)**
for PR 21203 at commit
[`9558ad7`](https://github.com/apache/spark/commit/95
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21196
**[Test build #89984 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89984/testReport)**
for PR 21196 at commit
[`4d1775c`](https://github.com/apache/spark/commit/4d
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21186
Hi, @gatorsmile .
Do you think we need to split this into two separate PRs? If you want, I
will split this.
- [SPARK-22279](https://issues.apache.org/jira/browse/SPARK-24112) Enable
co
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/20261
LGTM
Will merge after fresh tests
Thanks @WeichenXu123 and @smurakozi !
---
-
To unsubscribe, e-mail: reviews-unsubscr.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20261
**[Test build #4164 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4164/testReport)**
for PR 20261 at commit
[`d52b8ca`](https://github.com/apache/spark/commit/d
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21186
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21203
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
1 - 100 of 567 matches
Mail list logo
