Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21546
**[Test build #95397 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95397/testReport)**
for PR 21546 at commit
[`ffb47cb`](https://github.com/apache/spark/commit/f
Github user seancxmao commented on the issue:
https://github.com/apache/spark/pull/22262
@dongjoon-hyun @cloud-fan @gatorsmile Could you please kindly review this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@sp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22112
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-unified/2661/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22263
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-unified/2660/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22112
**[Test build #95404 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95404/testReport)**
for PR 22112 at commit
[`a4e6639`](https://github.com/apache/spark/commit/a4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22112
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/22263
**[Test build #95403 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95403/testReport)**
for PR 22263 at commit
[`9e058d1`](https://github.com/apache/spark/commit/9e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22263
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22112
I tried the failed test cases locally and can't reproduce, seems they are
flaky. Let me retest.
---
-
To unsubscribe, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22112
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
GitHub user wangyum opened a pull request:
https://github.com/apache/spark/pull/22263
[SPARK-25269][SQL] SQL interface support specify StorageLevel when cache
table
## What changes were proposed in this pull request?
SQL interface support specify `StorageLevel` when cache t
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22262
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/22262
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/22262
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user seancxmao opened a pull request:
https://github.com/apache/spark/pull/22262
[SPARK-25175][SQL] Field resolution should fail if there is ambiguity for
ORC native reader
## What changes were proposed in this pull request?
This PR aims to make ORC data source native imp
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22259
**[Test build #4297 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4297/testReport)**
for PR 22259 at commit
[`d99cfa6`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22112
**[Test build #4298 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4298/testReport)**
for PR 22112 at commit
[`a4e6639`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22112
**[Test build #4299 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4299/testReport)**
for PR 22112 at commit
[`a4e6639`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22258
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95396/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22258
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/22258
**[Test build #95396 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95396/testReport)**
for PR 22258 at commit
[`1503974`](https://github.com/apache/spark/commit/1
Github user npoberezkin commented on the issue:
https://github.com/apache/spark/pull/22255
@dbtsai Hello, I'm sorry for asking you directly, but for some reason
jenkins did not generate message: "Can one of the admins verify this patch?".
I just saw that you've reviewed some other PR
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21859#discussion_r213560104
--- Diff: core/src/main/scala/org/apache/spark/Partitioner.scala ---
@@ -138,7 +138,8 @@ class RangePartitioner[K : Ordering : ClassTag, V](
parti
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22184
Since we are not going to backport the PR to 2.3, do we still need this
migration guide?
Strictly speaking, we do have a behavior change here: hive table is always
case insensitive and we
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22250#discussion_r213557635
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1190,11 +1191,13 @@ object PushPredicateThroughJoin
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20637#discussion_r213556478
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ExpressionEvalHelperSuite.scala
---
@@ -35,6 +35,24 @@ class ExpressionEv
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/22186
The fix itself LGTM, but I don't think this could solve the STS shutdown
hook conflict problem with Hadoop.
---
-
To unsubscri
Github user yucai commented on a diff in the pull request:
https://github.com/apache/spark/pull/22197#discussion_r213551202
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilters.scala
---
@@ -350,25 +356,38 @@ private[parquet] class Pa
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21546
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/21546
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95390/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21546
**[Test build #95390 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95390/testReport)**
for PR 21546 at commit
[`ffb47cb`](https://github.com/apache/spark/commit/f
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21860#discussion_r213547480
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/HashAggregateExec.scala
---
@@ -853,33 +853,48 @@ case class HashAggregateExec
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/2
+1 on @rdblue 's idea. One point is, we should use
`ColumnarBatchScan.supportsBatch` to indicate columnar scan or not, instead of
asking the RDD to report it.
---
--
Github user ivoson commented on a diff in the pull request:
https://github.com/apache/spark/pull/22252#discussion_r213546414
--- Diff: docs/configuration.md ---
@@ -152,7 +152,7 @@ of the most common options to set are:
spark.driver.memory
1g
-Amount of
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/21546
@HyukjinKwon I redid the benchmarks for `toPandas` with the current code
and updated the description. It's not a huge speedup now, but definitely does
improve some. I'll also followup with anoth
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21669
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-unified/2659/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21669
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/21669
Kubernetes integration test status failure
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/2659/
---
--
Github user ivoson commented on a diff in the pull request:
https://github.com/apache/spark/pull/22252#discussion_r213544524
--- Diff: docs/configuration.md ---
@@ -152,7 +152,7 @@ of the most common options to set are:
spark.driver.memory
1g
-Amount of
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22197#discussion_r213544078
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetFilters.scala
---
@@ -350,25 +356,38 @@ private[parquet] clas
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21669
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/2659/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
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/22138
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95394/
Test PASSed.
---
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22232#discussion_r213543748
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileSourceSuite.scala
---
@@ -0,0 +1,55 @@
+/*
+ * Licensed to the Apac
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #95394 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95394/testReport)**
for PR 22138 at commit
[`ba576e8`](https://github.com/apache/spark/commit/b
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22261
What is `.1` in the title `[SPARK-25248.1]`?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/19691
@HyukjinKwon can you trigger again?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21912#discussion_r213542353
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -385,107 +385,120 @@ case class MapEntr
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22246
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21669
**[Test build #95402 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95402/testReport)**
for PR 21669 at commit
[`719b059`](https://github.com/apache/spark/commit/71
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21912
LGTM except a few comments
---
-
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/22261
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/22261
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95399/
Test PASSed.
---
Github user dbtsai commented on the issue:
https://github.com/apache/spark/pull/22246
Thanks all for reviewing. Merged into master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional co
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22261
**[Test build #95399 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95399/testReport)**
for PR 22261 at commit
[`afb50ee`](https://github.com/apache/spark/commit/a
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21912#discussion_r213541643
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -385,107 +385,120 @@ case class MapEntr
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/7
Also, you need to update split in python and R.
---
-
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/21912#discussion_r213540868
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -1490,6 +1423,63 @@ object CodeGenerat
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/7#discussion_r213540571
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
---
@@ -229,33 +229,59 @@ case class RLike(left: Exp
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21912#discussion_r213540562
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -1490,6 +1423,63 @@ object CodeGenerat
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21912#discussion_r213540440
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -1314,9 +1314,9 @@ class Collecti
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22146
Build finished. Test PASSed.
---
-
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/22146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95392/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21912#discussion_r213540247
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -473,13 +473,13 @@ public static UnsafeArrayDa
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22146
**[Test build #95392 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95392/testReport)**
for PR 22146 at commit
[`f3b6082`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22198
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/22198
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95395/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22198
**[Test build #95395 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95395/testReport)**
for PR 22198 at commit
[`cc0cd4f`](https://github.com/apache/spark/commit/c
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/7#discussion_r213539793
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
---
@@ -229,33 +229,59 @@ case class RLike(left: Exp
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21912#discussion_r213539798
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -1490,6 +1423,63 @@ object CodeGenerat
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/7#discussion_r213539767
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
---
@@ -229,33 +229,59 @@ case class RLike(left: Exp
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21912#discussion_r213539696
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/UnsafeArrayData.java
---
@@ -473,13 +473,13 @@ public static UnsafeArrayDa
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22260
**[Test build #95401 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95401/testReport)**
for PR 22260 at commit
[`626f265`](https://github.com/apache/spark/commit/62
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22112
**[Test build #95400 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95400/testReport)**
for PR 22112 at commit
[`a4e6639`](https://github.com/apache/spark/commit/a4
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22260
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user mengxr commented on the issue:
https://github.com/apache/spark/pull/22247
@squito Thanks for the refactor!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mai
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22112
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-unified/2658/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22112
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22112
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/7#discussion_r213538271
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
---
@@ -229,33 +229,59 @@ case class RLike(left: Exp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22112
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/22112
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95386/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22112
**[Test build #95386 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95386/testReport)**
for PR 22112 at commit
[`a4e6639`](https://github.com/apache/spark/commit/a
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22259
LGTM. How did you work around the type tag not found issue?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user xuejianbest commented on a diff in the pull request:
https://github.com/apache/spark/pull/22048#discussion_r213537923
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -294,23 +294,25 @@ class Dataset[T] private[sql](
// We set a minimu
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/7#discussion_r213538010
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
---
@@ -229,33 +229,59 @@ case class RLike(left: Exp
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22048#discussion_r213537514
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -294,23 +294,29 @@ class Dataset[T] private[sql](
// We set a minimum col
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22261
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/22246
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95393/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22246
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/22240#discussion_r213537490
--- Diff: core/src/main/scala/org/apache/spark/BarrierTaskContext.scala ---
@@ -68,7 +74,7 @@ class BarrierTaskContext(
*
* CAUTION! In a barr
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22048#discussion_r213537463
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -294,23 +294,29 @@ class Dataset[T] private[sql](
// We set a minimum col
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22261
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-unified/2657/
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/22048#discussion_r213537424
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -294,23 +294,29 @@ class Dataset[T] private[sql](
// We set a minimum col
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22246
**[Test build #95393 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95393/testReport)**
for PR 22246 at commit
[`bab5947`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22261
**[Test build #95399 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95399/testReport)**
for PR 22261 at commit
[`afb50ee`](https://github.com/apache/spark/commit/af
GitHub user mengxr opened a pull request:
https://github.com/apache/spark/pull/22261
[SPARK-25248.1][PYSPARK] update barrier Python API
## What changes were proposed in this pull request?
I made one pass over the Python APIs for barrier mode and updated them to
match the Sc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22209
Merged build finished. Test FAILed.
---
-
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/22260
Can you add `[SQL][MINOR]` in the title? Also, can you narrow down the
tittle cuz it is a little obscure.
---
-
To unsubscribe, e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22209
**[Test build #95382 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95382/testReport)**
for PR 22209 at commit
[`76f1801`](https://github.com/apache/spark/commit/7
1 - 100 of 693 matches
Mail list logo