Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21007
@BryanCutler @ueshin
add test in
https://github.com/apache/spark/blob/master/python/pyspark/tests.py?
---
-
To
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21006
**[Test build #89091 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89091/testReport)**
for PR 21006 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21006
Jenkins, ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21006
@susanxhuynh
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/20611
@sujith71955 any updates?
---
-
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/20757
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/2126/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20757
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20757
**[Test build #89090 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89090/testReport)**
for PR 20757 at commit
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180319251
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -190,28 +160,114 @@ case class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180318978
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -190,28 +160,114 @@ case class
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/20913
@mshtelma Would you please update the branch? seems there's something wrong
with the commits.
---
-
To unsubscribe, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/20757
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180317825
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -190,28 +160,114 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21020
**[Test build #89089 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89089/testReport)**
for PR 21020 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21020
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89089/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21020
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180317210
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -190,28 +160,114 @@ case class
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21011#discussion_r180317060
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +288,173 @@ case class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180316279
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -190,28 +160,114 @@ case class
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20989
hi there - you should always open PR on master. we could backport to "older
releases" as needed, or ask you to open a separate PR that is based on
branch-2.3 (like this one you have now)
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21019
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21019
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89085/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20757
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20757
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89084/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21021
**[Test build #89088 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89088/testReport)**
for PR 21021 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21019
**[Test build #89085 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89085/testReport)**
for PR 21019 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89088/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20757
**[Test build #89084 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89084/testReport)**
for PR 20757 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21020
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21020
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/2125/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21020
**[Test build #89089 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89089/testReport)**
for PR 21020 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21020
> thats fine but I thought Jenkins is running 6.0.1? or maybe only on some
boxes
I am not sure of the specific version but I assume it's older version ..
---
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21020#discussion_r180313623
--- Diff: docs/README.md ---
@@ -22,10 +22,13 @@ $ sudo gem install jekyll jekyll-redirect-from
pygments.rb
$ sudo pip install Pygments
#
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21010#discussion_r180313214
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -2108,35 +2132,52 @@ case class
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/21020#discussion_r180312977
--- Diff: docs/README.md ---
@@ -22,10 +22,13 @@ $ sudo gem install jekyll jekyll-redirect-from
pygments.rb
$ sudo pip install Pygments
#
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21010#discussion_r180312765
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -2108,35 +2132,52 @@ case class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21010#discussion_r180312702
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -2108,35 +2132,52 @@ case class
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180312180
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -190,28 +160,114 @@ case class
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180311923
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -190,28 +160,114 @@ case class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21010#discussion_r180311991
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -2050,33 +2058,49 @@ case class
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21011#discussion_r180311653
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -287,3 +288,173 @@ case class
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21011#discussion_r180311581
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -413,6 +413,29 @@ class DataFrameFunctionsSuite extends
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180311343
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -190,28 +160,114 @@ case class
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180310842
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -324,6 +324,30 @@ class DataFrameFunctionsSuite extends
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21010#discussion_r180310931
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -2022,6 +2022,8 @@ case class Encode(value:
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180310741
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -190,28 +160,114 @@ case class
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180310339
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -116,47 +116,17 @@ case class
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21021
@HyukjinKwon thanks, fixed it.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180310089
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -116,47 +116,17 @@ case class
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21021
Seems JIRA number wrong ..
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21021#discussion_r180309744
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -190,28 +160,114 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
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/2124/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21021
**[Test build #89088 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89088/testReport)**
for PR 21021 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21017
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89086/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21017
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user kiszk opened a pull request:
https://github.com/apache/spark/pull/21021
[SPARK-23916][SQL] Add array_sort function
## What changes were proposed in this pull request?
The PR adds the SQL function `array_sort`. The behavior of the function is
based on Presto's
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21017
**[Test build #89086 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89086/testReport)**
for PR 21017 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21009#discussion_r180307648
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -426,15 +426,7 @@ case class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21009#discussion_r180306495
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/datetimeExpressions.scala
---
@@ -456,6 +448,49 @@ case class
701 - 761 of 761 matches
Mail list logo