Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118402369
[Test build #36512 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36512/consoleFull)
for PR 7207 at commit
Github user spirom commented on a diff in the pull request:
https://github.com/apache/spark/pull/7207#discussion_r33876368
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/UDFSuite.scala ---
@@ -82,6 +83,48 @@ class UDFSuite extends QueryTest {
assert(ctx.sql(SELECT
Github user spirom commented on a diff in the pull request:
https://github.com/apache/spark/pull/7207#discussion_r33876378
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/UDFSuite.scala ---
@@ -82,6 +83,48 @@ class UDFSuite extends QueryTest {
assert(ctx.sql(SELECT
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118402048
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118402061
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 user asfgit closed the pull request at:
https://github.com/apache/spark/pull/7207
---
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 user rxin commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118454038
Thanks - I've merged this in 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118411628
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118411589
[Test build #36512 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36512/console)
for PR 7207 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118248348
Thanks - looks pretty good for the first patch!
There are just some minor style issues.
---
If your project is set up for it, you can reply to this email and
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118246915
Can one of the admins verify this patch?
---
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 user sarutak commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118250753
ok to test.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118251660
[Test build #36484 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36484/consoleFull)
for PR 7207 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118251829
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 user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118251827
[Test build #36484 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36484/console)
for PR 7207 at commit
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118248560
Jenkins, ok to test.
---
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
Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/7207#discussion_r33843582
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/UDFSuite.scala ---
@@ -82,6 +83,48 @@ class UDFSuite extends QueryTest {
assert(ctx.sql(SELECT
GitHub user spirom opened a pull request:
https://github.com/apache/spark/pull/7207
[SPARK-8810] [SQL] Added several UDF unit tests for Spark SQL
One test for each of the GROUP BY, WHERE and HAVING clauses, and one that
combines all three with an additional UDF in the SELECT.
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7207#discussion_r33842367
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/UDFSuite.scala ---
@@ -82,6 +83,48 @@ class UDFSuite extends QueryTest {
assert(ctx.sql(SELECT
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7207#discussion_r33842348
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/UDFSuite.scala ---
@@ -82,6 +83,48 @@ class UDFSuite extends QueryTest {
assert(ctx.sql(SELECT
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/7207#discussion_r33842386
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/UDFSuite.scala ---
@@ -82,6 +83,48 @@ class UDFSuite extends QueryTest {
assert(ctx.sql(SELECT
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118251369
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 user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/7207#issuecomment-118251389
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
23 matches
Mail list logo