Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20492
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20483
Thank you for adding this. I will look at the performance regression.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apach
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/20274
The [Pre-build
spark](https://dist.apache.org/repos/dist/dev/spark/v2.3.0-rc2-bin/) contains
`kubernetes-model-2.0.0.jar`. but the below command will not contain this jar:
```
./dev/make-dis
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20400
**[Test build #87061 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87061/testReport)**
for PR 20400 at commit
[`f82c7d1`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20400
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87061/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20400
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/13599
ping @holdenk @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: re
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/20274
An interesting discovery:
if `SPARK_HOME/jars` missing `kubernetes-model-2.0.0.jar`, the silent mode
is broken.
---
-
To uns
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20492
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87059/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20492
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/20492
**[Test build #87059 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87059/testReport)**
for PR 20492 at commit
[`732cf9e`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20400
**[Test build #87061 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87061/testReport)**
for PR 20400 at commit
[`f82c7d1`](https://github.com/apache/spark/commit/f8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20400
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/584/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20400
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user huaxingao commented on a diff in the pull request:
https://github.com/apache/spark/pull/20400#discussion_r165893442
--- Diff: python/pyspark/sql/window.py ---
@@ -208,20 +236,27 @@ def rangeBetween(self, start, end):
and "5" means the five off after the curr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20487
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/20487
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87057/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20487
**[Test build #87057 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87057/testReport)**
for PR 20487 at commit
[`a0e4b16`](https://github.com/apache/spark/commit/a
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/20504
Thanks @hvanhovell , the major changes is `SQLQueryTestSuite.scala`:
```scala
private def listTestCases(): Seq[TestCase] = {
-listFilesRecursively(new File(inputFilePath)).map { fi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20505
**[Test build #87060 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87060/testReport)**
for PR 20505 at commit
[`d216760`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20505
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87060/
Test FAILed.
---
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20502#discussion_r165890435
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -179,7 +179,7 @@ private[ui] abstract class Executio
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20505
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/20504
@wangyum this is a PR of 24K lines! How are we going to review this
properly? Can you provide some guidelines (major changes etc)?
---
--
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20505
**[Test build #87060 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87060/testReport)**
for PR 20505 at commit
[`d216760`](https://github.com/apache/spark/commit/d2
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/20505
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/20505#discussion_r165889177
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLImplicits.scala
---
@@ -165,11 +165,15 @@ abstract class SQLImplicits extends
LowPrioritySQLIm
Github user advancedxy commented on the issue:
https://github.com/apache/spark/pull/17422
All right then, I will take it over. Of course the credit should go to
@noodle-fb.
We can discuss whether this behaviour is desirable or not in the JIRA or
the new PR.
---
---
Github user advancedxy commented on the issue:
https://github.com/apache/spark/pull/20449
ping @cloud-fan and @jiangxb1987.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20487#discussion_r165884501
--- Diff: pom.xml ---
@@ -185,6 +185,10 @@
2.8
1.8
1.0.0
+
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20499#discussion_r165883566
--- Diff: python/pyspark/sql/tests.py ---
@@ -2186,7 +2186,7 @@ def test_replace(self):
# replace with subset specified with one column replaced,
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20499#discussion_r165884365
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1587,6 +1600,52 @@ def replace(self, to_replace, value=None,
subset=None):
|null| null|null|
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20499#discussion_r165883051
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1557,6 +1557,9 @@ def replace(self, to_replace, value=None,
subset=None):
For example, if
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20487#discussion_r165884027
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1923,6 +1923,9 @@ def toPandas(self):
02 Alice
15Bob
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20487#discussion_r165883842
--- Diff: pom.xml ---
@@ -185,6 +185,10 @@
2.8
1.8
1.0.0
+
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20487#discussion_r165883676
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1923,6 +1923,9 @@ def toPandas(self):
02 Alice
15Bob
"""
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20487#discussion_r165879157
--- Diff: pom.xml ---
@@ -185,6 +185,10 @@
2.8
1.8
1.0.0
+
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20504
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87054/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20504
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/20504
**[Test build #87054 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87054/testReport)**
for PR 20504 at commit
[`41ba497`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18555
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/18555
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87053/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18555
**[Test build #87053 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87053/testReport)**
for PR 18555 at commit
[`a0efb41`](https://github.com/apache/spark/commit/a
Github user cenyuhai commented on a diff in the pull request:
https://github.com/apache/spark/pull/16099#discussion_r165878592
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkExecuteStatementOperation.scala
---
@@ -241,6 +241,8 @@ private
Github user cenyuhai commented on the issue:
https://github.com/apache/spark/pull/16099
@gatorsmile two years passed... I don't know what to say.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20164
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87058/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20164
**[Test build #87058 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87058/testReport)**
for PR 20164 at commit
[`f9a922f`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20164
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/20473
LGTM.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apach
Github user cenyuhai commented on a diff in the pull request:
https://github.com/apache/spark/pull/16099#discussion_r165877094
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkExecuteStatementOperation.scala
---
@@ -241,6 +241,8 @@ private
Github user BruceXu1991 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16099#discussion_r165876866
--- Diff:
sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/SparkExecuteStatementOperation.scala
---
@@ -241,6 +241,8 @@ priv
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20502#discussion_r165876103
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -179,7 +179,7 @@ private[ui] abstract class ExecutionT
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20373
Will merge this one in few days if there's no more comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20492
**[Test build #87059 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87059/testReport)**
for PR 20492 at commit
[`732cf9e`](https://github.com/apache/spark/commit/73
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20492
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/583/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20492
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user sitalkedia commented on a diff in the pull request:
https://github.com/apache/spark/pull/20492#discussion_r165874317
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeSorterSpillReader.java
---
@@ -77,7 +77,7 @@ public UnsafeSorterSpillRea
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20164
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/582/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20164
**[Test build #87058 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87058/testReport)**
for PR 20164 at commit
[`f9a922f`](https://github.com/apache/spark/commit/f9
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20473
Will merge this one if there's no more comments in few days.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20487
**[Test build #87057 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87057/testReport)**
for PR 20487 at commit
[`a0e4b16`](https://github.com/apache/spark/commit/a0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20164
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20487#discussion_r165873671
--- Diff: python/setup.py ---
@@ -100,6 +100,11 @@ def _supports_symlinks():
file=sys.stderr)
exit(-1)
+# If yo
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20487#discussion_r165873582
--- Diff: python/pyspark/sql/tests.py ---
@@ -2794,7 +2792,6 @@ def count_bucketed_cols(names,
table="pyspark_bucket"):
def _to_pandas(se
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20487
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/581/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20487
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20487#discussion_r165873632
--- Diff: pom.xml ---
@@ -185,6 +185,10 @@
2.8
1.8
1.0.0
+
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/20164
@WeichenXu123 Yes, my concern is that it is confusing if the transform
failure is caused by column conflict by a âinvisibleâ column.
@srowen Agree that it is not perfect if we alter
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20502#discussion_r165872846
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/ui/AllExecutionsPage.scala
---
@@ -179,7 +179,7 @@ private[ui] abstract class ExecutionTabl
Github user lucio-yz commented on a diff in the pull request:
https://github.com/apache/spark/pull/20472#discussion_r165872418
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/RandomForest.scala ---
@@ -1001,11 +1002,22 @@ private[spark] object RandomForest extends Logg
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20473
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87056/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20473
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/20473
**[Test build #87056 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87056/testReport)**
for PR 20473 at commit
[`78f5879`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20473
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87055/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20473
**[Test build #87055 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87055/testReport)**
for PR 20473 at commit
[`fe2943e`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20473
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/20473
**[Test build #87056 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87056/testReport)**
for PR 20473 at commit
[`78f5879`](https://github.com/apache/spark/commit/78
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20473
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/580/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20473
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20473#discussion_r165869377
--- Diff: python/run-tests.py ---
@@ -151,6 +152,67 @@ def parse_opts():
return opts
+def _check_dependencies(python_exec, modul
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20473
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/579/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20473
**[Test build #87055 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87055/testReport)**
for PR 20473 at commit
[`fe2943e`](https://github.com/apache/spark/commit/fe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20473
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/20504
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/578/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20504
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/20504
**[Test build #87054 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87054/testReport)**
for PR 20504 at commit
[`41ba497`](https://github.com/apache/spark/commit/41
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20504
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/20504
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87052/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20504
**[Test build #87052 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87052/testReport)**
for PR 20504 at commit
[`dd8531d`](https://github.com/apache/spark/commit/d
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/20504
After SPARK-21646, `hive/binaryComparison.sql.out`,
`hive/decimalPrecision.sql.out` and `hive/promoteStrings.sql.out` seems like
this:
https://github.com/wangyum/spark/commit/927f6e86712ec4da4d58db
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18555
**[Test build #87053 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87053/testReport)**
for PR 18555 at commit
[`a0efb41`](https://github.com/apache/spark/commit/a0
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20487#discussion_r165865310
--- Diff: python/pyspark/sql/utils.py ---
@@ -115,18 +115,30 @@ def toJArray(gateway, jtype, arr):
def require_minimum_pandas_version():
""
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20487#discussion_r165865284
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1923,6 +1923,9 @@ def toPandas(self):
02 Alice
15Bob
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20505
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/20505
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user michalsenkyr opened a pull request:
https://github.com/apache/spark/pull/20505
[SPARK-23251][SQL] Add checks for collection element Encoders
Implicit methods of `SQLImplicits` providing Encoders for collections did
not check for
Encoders for their elements. This resu
Github user eyalfa commented on a diff in the pull request:
https://github.com/apache/spark/pull/19054#discussion_r165861581
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/EnsureRequirements.scala
---
@@ -220,45 +220,99 @@ case class EnsureRequirements(c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20504
**[Test build #87052 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87052/testReport)**
for PR 20504 at commit
[`dd8531d`](https://github.com/apache/spark/commit/dd
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20504
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/577/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20504
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
1 - 100 of 187 matches
Mail list logo