Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20387#discussion_r169239000
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -17,17 +17,80 @@
package org
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20387
thanks, merging to master!
---
-
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/20387
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20621
thanks @cloud-fan. Sorry, since this seems a bug to me, why this wasn't
backported to branch-2.3 too? Thanks.
---
-
To unsubscr
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20616#discussion_r169248859
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/FileStreamSink.scala
---
@@ -42,9 +42,13 @@ object FileStreamSink extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20616
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/20616
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/964/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20616
**[Test build #87550 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87550/testReport)**
for PR 20616 at commit
[`27188e7`](https://github.com/apache/spark/commit/27
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/20481
I didn't know InMemoryStore doesn't have any index... Just saw the UI is
very slow on a large cluster and it causes read timeout.
---
--
Github user gerashegalov commented on a diff in the pull request:
https://github.com/apache/spark/pull/20327#discussion_r169257050
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
@@ -744,7 +744,9 @@ object SparkSubmit extends CommandLineUtils with
Loggin
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20481
IIUC we use index a lot in the UI code, @vanzin is it possible to support
index for in-memory kv store?
---
-
To unsubscribe,
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/20621
it's not a very serious bug, I'd like to hold it until 2.3 is released. We
may have it in 2.3.1
---
-
To unsubscribe, e-mail:
GitHub user gaborgsomogyi opened a pull request:
https://github.com/apache/spark/pull/20639
[SPARK-23288][SS] Fix output metrics with parquet sink
## What changes were proposed in this pull request?
Output metrics were not filled when parquet sink used.
This PR fixe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20639
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/20639
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user misutoth commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169287270
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -262,6 +273,11 @@ case class Cos(child: Expre
Github user misutoth commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169287876
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -521,7 +542,15 @@ case class Signum(child: Ex
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20424
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20424
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user misutoth commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169291756
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -1313,131 +1313,165 @@ object functions {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20618
**[Test build #87551 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87551/testReport)**
for PR 20618 at commit
[`8550a27`](https://github.com/apache/spark/commit/85
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20616
**[Test build #87550 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87550/testReport)**
for PR 20616 at commit
[`27188e7`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20616
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87550/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20616
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user sujith71955 commented on the issue:
https://github.com/apache/spark/pull/20611
retest please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169299563
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -252,7 +255,14 @@ case class Ceil(child: E
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169300055
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -262,6 +272,10 @@ case class Cos(child: Ex
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169299115
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -2005,71 +1967,63 @@ object functions {
def signum(columnName: Strin
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19222
**[Test build #87552 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87552/testReport)**
for PR 19222 at commit
[`1fa47a8`](https://github.com/apache/spark/commit/1f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19222
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/19222
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/965/
Test
GitHub user IgorBerman opened a pull request:
https://github.com/apache/spark/pull/20640
[SPARK-19755][Mesos] Blacklist is always active for
MesosCoarseGrainedSchedulerBackend
## What changes were proposed in this pull request?
This is another variant of https://github.com/apach
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20640
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/20640
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169305100
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -262,6 +272,10 @@ case class Cos(child: Ex
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169305384
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -548,7 +579,14 @@ case class Sqrt(child: E
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169305293
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -539,7 +563,14 @@ case class Sinh(child: E
Github user misutoth commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169311232
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -2005,71 +1967,63 @@ object functions {
def signum(columnName: String):
Github user misutoth commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169311993
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -252,7 +255,14 @@ case class Ceil(child: Expr
Github user misutoth commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169312315
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -262,6 +272,10 @@ case class Cos(child: Expre
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20627
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/20627
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/966/
Test
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20627
**[Test build #87553 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87553/testReport)**
for PR 20627 at commit
[`6b379ef`](https://github.com/apache/spark/commit/6b
Github user misutoth commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169313008
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -262,6 +272,10 @@ case class Cos(child: Expre
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/20612#discussion_r169313393
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -1226,14 +1226,24 @@ class CodegenContext
Github user misutoth commented on a diff in the pull request:
https://github.com/apache/spark/pull/20618#discussion_r169313329
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala
---
@@ -548,7 +579,14 @@ case class Sqrt(child: Expr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20618
**[Test build #87554 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87554/testReport)**
for PR 20618 at commit
[`40da998`](https://github.com/apache/spark/commit/40
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/20043#discussion_r169317284
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/BoundAttribute.scala
---
@@ -75,7 +75,7 @@ case class BoundReference(ordinal
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20627
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87553/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20627
**[Test build #87553 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87553/testReport)**
for PR 20627 at commit
[`6b379ef`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20627
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/20612
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/20612
**[Test build #87555 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87555/testReport)**
for PR 20612 at commit
[`eed9dcd`](https://github.com/apache/spark/commit/ee
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20612
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/967/
Test
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20571
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20618
@misutoth, I am sorry. I missed
https://github.com/apache/spark/pull/20618#discussion_r169291756 and discussion
in the JIRA.
@srowen, I was thinking about having the same description pa
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20571
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20640
Jenkins, ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20640
**[Test build #87556 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87556/testReport)**
for PR 20640 at commit
[`ca4eba2`](https://github.com/apache/spark/commit/ca
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20640
@IgorBerman I actually think that
https://github.com/apache/spark/pull/17619 is the right approach. As @timout
pointed out on that one, this functionality doesn't need to be covered in mesos
specif
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20618
**[Test build #87551 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87551/testReport)**
for PR 20618 at commit
[`8550a27`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20618
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/20640
**[Test build #87556 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87556/testReport)**
for PR 20640 at commit
[`ca4eba2`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20640
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87556/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20618
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87551/
Test PASSed.
---
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20618
Consistency is good. You're suggesting updating Python, R separately? If
it's a big change, I could see breaking it up, but could also imagine adjusting
that here. I am OK with getting this in first
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20640
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user IgorBerman commented on the issue:
https://github.com/apache/spark/pull/20640
@squito In general I agree with you, however current status of hardcoded
max 2 failures is blocking, so I've created simple fix that will demand less
integration testing. I can close it.
I th
GitHub user krcz opened a pull request:
https://github.com/apache/spark/pull/20641
[SPARK-23464][MESOS] Fix mesos cluster scheduler options double-escaping
## What changes were proposed in this pull request?
Don't enclose --conf option value with "", as they are already esca
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20641
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/20641
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20618
Maybe I am too much caring about this and It guess it wouldn't be a big
change; however, I was thinking there might be subtle nits, like, referring
`java.lang.Math*` in Python side and R side wh
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19222
**[Test build #87552 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87552/testReport)**
for PR 19222 at commit
[`1fa47a8`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19222
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/19222
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87552/
Test FAILed.
---
GitHub user AtulKumVerma opened a pull request:
https://github.com/apache/spark/pull/20642
i m not able to open Spark UI on local using localhost:4040
20-02-18 20:54:24:118 [WARN] org.spark_project.jetty.server.HttpChannel
/jobs/ org.spark_project.jetty.server.HttpChannel.handle(Htt
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20642
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/20642
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20640
I understand if you want to do something like this for yourself to unblock,
but I think I'm -1 on merging this because of adding more configs just for a
stopgap.
but I think we agree on the
Github user IgorBerman commented on the issue:
https://github.com/apache/spark/pull/20640
@squito ok, I can do this small fix to the PR of @timout. How can I update
his PR/his code? Or I can use this one? WDYT?
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20618
**[Test build #87554 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87554/testReport)**
for PR 20618 at commit
[`40da998`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20618
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87554/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20618
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/20472
@srowen Can you trigger the tests?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user misutoth commented on the issue:
https://github.com/apache/spark/pull/20618
Looking back I guess we can expect a couple of comments on R and Python
side too, though I will target a lower number of them. :) So I am a little bit
favoring moving functions.* update into a sepa
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20472
Jenkins, add to whitelist
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20472
**[Test build #4101 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4101/testReport)**
for PR 20472 at commit
[`1716acc`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20472
**[Test build #4101 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4101/testReport)**
for PR 20472 at commit
[`1716acc`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20472
**[Test build #87557 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87557/testReport)**
for PR 20472 at commit
[`1716acc`](https://github.com/apache/spark/commit/17
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20472
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87557/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20472
**[Test build #87557 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87557/testReport)**
for PR 20472 at commit
[`1716acc`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20472
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20618
@HyukjinKwon I think it's fine to refer to `java.lang.Math` even from
Python, R documents. It's just expressing the API's behavior, and that contract
is the same regardless of the language. (Or is th
Github user IgorBerman commented on the issue:
https://github.com/apache/spark/pull/20640
@squito updated
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20640
**[Test build #87558 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87558/testReport)**
for PR 20640 at commit
[`2240259`](https://github.com/apache/spark/commit/22
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/20387
Thanks for all your help getting this committed, @cloud-fan!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/20642
@AtulKumVerma please close this PR. You can ask questions in the user list
or on stack overflow.
---
-
To unsubscribe, e-mail
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20640
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87558/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20640
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/20640
**[Test build #87558 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87558/testReport)**
for PR 20640 at commit
[`2240259`](https://github.com/apache/spark/commit/2
1 - 100 of 321 matches
Mail list logo