[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-10-04 Thread navis
Github user navis closed the pull request at: https://github.com/apache/spark/pull/8893 --- 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

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-10-01 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-144707576 It looks like @chenghao-intel is farther along towards a simpler fix in https://github.com/apache/spark/pull/8933/files Do you mind closing this PR? --- If your

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-10-01 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-144814604 @navis Since #8933 is merged, do you mind to close this one? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread navis
Github user navis commented on a diff in the pull request: https://github.com/apache/spark/pull/8893#discussion_r40290254 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala --- @@ -72,6 +72,18 @@ abstract class

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142833221 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

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142833210 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

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142833589 [Test build #42965 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42965/consoleFull) for PR 8893 at commit

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142855821 [Test build #42965 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42965/console) for PR 8893 at commit

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142855962 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142855957 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

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread ssimeonov
Github user ssimeonov commented on a diff in the pull request: https://github.com/apache/spark/pull/8893#discussion_r40337580 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala --- @@ -52,11 +52,12 @@ abstract class

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-143106979 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

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-143106960 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

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-143108350 [Test build #43003 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43003/consoleFull) for PR 8893 at commit

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread navis
Github user navis commented on a diff in the pull request: https://github.com/apache/spark/pull/8893#discussion_r40388170 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala --- @@ -52,11 +52,12 @@ abstract class

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-143108560 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-143108557 [Test build #43003 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43003/console) for PR 8893 at commit

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-143108559 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

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142780709 [Test build #42949 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42949/consoleFull) for PR 8893 at commit

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-23 Thread ssimeonov
Github user ssimeonov commented on a diff in the pull request: https://github.com/apache/spark/pull/8893#discussion_r40279751 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/mathExpressions.scala --- @@ -72,6 +72,18 @@ abstract class

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142783279 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142783250 [Test build #42949 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42949/console) for PR 8893 at commit

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142783278 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

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-23 Thread navis
GitHub user navis opened a pull request: https://github.com/apache/spark/pull/8893 [SPARK-10724] [SQL] SQL's floor() returns DOUBLE This is a change in behavior from 1.4.1 where {{floor}} returns a BIGINT. {code} scala> sql("select floor(1)").printSchema root

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142780411 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

[GitHub] spark pull request: [SPARK-10724] [SQL] SQL's floor() returns DOUB...

2015-09-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8893#issuecomment-142780421 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