[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/5412 --- 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] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91105998 Ok I've merged it. --- 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] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91088775 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91088769 [Test build #29904 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29904/consoleFull) for PR 5412 at commit [`9098a31`](https://gith

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91077219 [Test build #29904 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29904/consoleFull) for PR 5412 at commit [`9098a31`](https://githu

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91041145 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91041133 [Test build #29890 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29890/consoleFull) for PR 5412 at commit [`8c7250a`](https://gith

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91036759 [Test build #29890 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29890/consoleFull) for PR 5412 at commit [`8c7250a`](https://githu

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91035987 [Test build #29888 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29888/consoleFull) for PR 5412 at commit [`82d0944`](https://gith

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91035997 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91035253 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91035233 [Test build #29887 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29887/consoleFull) for PR 5412 at commit [`0b03fbb`](https://gith

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91031683 [Test build #29888 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29888/consoleFull) for PR 5412 at commit [`82d0944`](https://githu

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-91030225 [Test build #29887 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29887/consoleFull) for PR 5412 at commit [`0b03fbb`](https://githu

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5412#discussion_r28010130 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/QueryTest.scala --- @@ -67,7 +67,7 @@ class QueryTest extends PlanTest { checkAnswer(df, Seq(expec

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/5412#discussion_r27958976 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveQuerySuite.scala --- @@ -237,7 +238,11 @@ class HiveQuerySuite extends HiveCompar

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/5412#discussion_r27958798 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/parquetSuites.scala --- @@ -887,7 +886,11 @@ abstract class ParquetPartitioningTest extends Que

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/5412#discussion_r27958632 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/InsertIntoHiveTableSuite.scala --- @@ -115,11 +115,36 @@ class InsertIntoHiveTableSuite extends

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/5412#discussion_r27953124 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala --- @@ -182,7 +182,10 @@ class SQLQuerySuite extends QueryTest with BeforeAndAft

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/5412#discussion_r27953098 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/QueryTest.scala --- @@ -67,7 +67,7 @@ class QueryTest extends PlanTest { checkAnswer(df, Seq(

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-90838975 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-90838967 [Test build #29846 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29846/consoleFull) for PR 5412 at commit [`f2f4348`](https://gith

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-90829633 [Test build #29846 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29846/consoleFull) for PR 5412 at commit [`f2f4348`](https://githu

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/5412#discussion_r27948025 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/plans/PlanTest.scala --- @@ -45,12 +45,13 @@ class PlanTest extends FunSuite { p

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/5412#discussion_r27947942 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisSuite.scala --- @@ -202,7 +204,7 @@ class AnalysisSuite extends FunSuite wi

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/5412#discussion_r27947864 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisSuite.scala --- @@ -202,7 +204,7 @@ class AnalysisSuite extends FunSui

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-90828468 [Test build #29842 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29842/consoleFull) for PR 5412 at commit [`ef4ec48`](https://gith

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-90828471 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-08 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/5412#issuecomment-90824824 [Test build #29842 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/29842/consoleFull) for PR 5412 at commit [`ef4ec48`](https://githu

[GitHub] spark pull request: [SPARK-6765] Fix test code style for SQL

2015-04-07 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/5412 [SPARK-6765] Fix test code style for SQL So we can turn style checker on for test code. You can merge this pull request into a Git repository by running: $ git pull https://github.com/rxin/spark t