Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113596972
retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this fea
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113596151
Looks like an unrelated failure.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113593440
[Test build #35287 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35287/console)
for PR 6721 at commit
[`cde33b5`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113593467
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113579177
[Test build #35287 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35287/consoleFull)
for PR 6721 at commit
[`cde33b5`](https://gith
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/6721
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enab
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113578905
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113578934
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113578138
LGTM, merging into master!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have t
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32848403
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/math.scala
---
@@ -205,6 +208,34 @@ case class ToRadians(child: Expression) ex
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113096382
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113096335
[Test build #35105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35105/console)
for PR 6721 at commit
[`07e1c8f`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113062135
[Test build #35105 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35105/consoleFull)
for PR 6721 at commit
[`07e1c8f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113061628
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113061601
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113061471
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113061458
[Test build #35098 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35098/console)
for PR 6721 at commit
[`0677f1a`](https://github.
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32703657
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/math.scala
---
@@ -51,16 +54,28 @@ abstract class LeafMathExpression(c: Double
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32703575
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/math.scala
---
@@ -51,16 +54,28 @@ abstract class LeafMathExpression(c: Double
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32702596
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/math.scala
---
@@ -51,16 +54,28 @@ abstract class LeafMathExpression(c: Double
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113053074
[Test build #35098 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/35098/consoleFull)
for PR 6721 at commit
[`0677f1a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113052885
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-113052869
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32702212
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/math.scala
---
@@ -51,16 +54,28 @@ abstract class LeafMathExpression(c: Double
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32702082
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/math.scala
---
@@ -51,16 +54,28 @@ abstract class LeafMathExpression(c: Double
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32653230
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/math.scala
---
@@ -51,16 +54,28 @@ abstract class LeafMathExpression(c: Double
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32651529
--- Diff: python/pyspark/sql/functions.py ---
@@ -230,6 +231,19 @@ def approxCountDistinct(col, rsd=None):
return Column(jc)
+@ignore_u
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32651940
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -882,6 +882,24 @@ object functions {
def atan2(l: Double, rightName: String
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32651872
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -17,6 +17,8 @@
package org.apache.spark.sql
+i
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32652438
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/math.scala
---
@@ -205,6 +217,26 @@ case class ToRadians(child: Expression) ex
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111984474
[Test build #34914 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34914/console)
for PR 6721 at commit
[`0cf20f2`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111984558
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111951645
[Test build #34914 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34914/consoleFull)
for PR 6721 at commit
[`0cf20f2`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111950169
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111950220
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111736050
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111736035
[Test build #34823 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34823/console)
for PR 6721 at commit
[`dea9c12`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111719367
[Test build #34823 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34823/consoleFull)
for PR 6721 at commit
[`dea9c12`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111719269
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111719265
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111255646
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111255608
[Test build #34698 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34698/console)
for PR 6721 at commit
[`d4f4774`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111211960
[Test build #34698 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34698/consoleFull)
for PR 6721 at commit
[`d4f4774`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111211457
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111211476
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-70178
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-70162
[Test build #34688 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34688/console)
for PR 6721 at commit
[`7a0196f`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-33742
[Test build #34688 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34688/consoleFull)
for PR 6721 at commit
[`7a0196f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-32593
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-10692
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-10014
retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this fea
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111072784
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111072746
[Test build #34677 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34677/console)
for PR 6721 at commit
[`7a0196f`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111037234
[Test build #34677 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34677/consoleFull)
for PR 6721 at commit
[`7a0196f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111037093
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111037072
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111036735
Build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does n
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111036734
[Test build #34676 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34676/console)
for PR 6721 at commit
[`a0a2d0f`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111036013
[Test build #34676 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34676/consoleFull)
for PR 6721 at commit
[`a0a2d0f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111035659
Build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this f
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-111035640
Build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have thi
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32139218
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala
---
@@ -163,6 +165,10 @@ class MathFunctionsSuite ext
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32139061
--- Diff: python/pyspark/sql/functions.py ---
@@ -136,6 +136,7 @@ def _():
'measured in radians.',
'bitwiseNOT': 'Computes bi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32139013
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala
---
@@ -122,6 +122,7 @@ object FunctionRegistry {
e
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32138987
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala
---
@@ -163,6 +165,10 @@ class MathFunctionsSuite exten
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32138877
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala
---
@@ -163,6 +165,10 @@ class MathFunctionsSuite exten
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32138809
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala
---
@@ -163,6 +165,10 @@ class MathFunctionsSuite exten
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32138774
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -90,4 +92,11 @@ class DataFrameFunctionsSuite extends QueryTest {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32138727
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -1299,6 +1299,23 @@ object functions {
*/
def toRadians(columnName: S
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110681938
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110681921
[Test build #34583 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34583/console)
for PR 6721 at commit
[`4cb764d`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110642510
[Test build #34583 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34583/consoleFull)
for PR 6721 at commit
[`4cb764d`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110642143
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110642158
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32074111
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/math.scala
---
@@ -17,24 +17,35 @@
package org.apache.spark.sql.catal
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32074025
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/MathFunctionsSuite.scala
---
@@ -31,11 +31,12 @@ class MathFunctionsSuite extend
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32074005
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -90,4 +92,10 @@ class DataFrameFunctionsSuite extends QueryTest {
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/6721#discussion_r32073874
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -1299,6 +1299,21 @@ object functions {
*/
def toRadians(columnName: S
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110443331
[Test build #34507 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34507/console)
for PR 6721 at commit
[`c0c3197`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110443349
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110401661
[Test build #34507 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34507/consoleFull)
for PR 6721 at commit
[`c0c3197`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110400939
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110400982
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110356083
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110356067
[Test build #34500 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34500/console)
for PR 6721 at commit
[`50e0c3b`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110318435
[Test build #34500 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34500/consoleFull)
for PR 6721 at commit
[`50e0c3b`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110317986
Merged build started.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6721#issuecomment-110317962
Merged build triggered.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not h
GitHub user viirya opened a pull request:
https://github.com/apache/spark/pull/6721
[SPARK-8207][SQL] Add math function bin
JIRA: https://issues.apache.org/jira/browse/SPARK-8207
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vi
89 matches
Mail list logo
