[GitHub] spark pull request: [SPARK-8608][SPARK-8609][SQL] reset mutable st...

2015-07-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7674#issuecomment-124968686 [Test build #38469 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/38469/consoleFull) for PR 7674 at commit [`e3814fb`](https://gith

[GitHub] spark pull request: [SPARK-8608][SPARK-8609][SQL] reset mutable st...

2015-07-26 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/7674#discussion_r35491410 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala --- @@ -825,11 +813,43 @@ class DataFrameSuite extends QueryTest with SQLTestUt

[GitHub] spark pull request: [SPARK-8608][SPARK-8609][SQL] reset mutable st...

2015-07-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7674#issuecomment-124974475 [Test build #38467 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/38467/console) for PR 7674 at commit [`8f57ede`](https://github.

[GitHub] spark pull request: [SPARK-8608][SPARK-8609][SQL] reset mutable st...

2015-07-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7674#issuecomment-124974493 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] spark pull request: [SPARK-8608][SPARK-8609][SQL] reset mutable st...

2015-07-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7674#issuecomment-124976451 [Test build #38469 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/38469/console) for PR 7674 at commit [`e3814fb`](https://github.

[GitHub] spark pull request: [SPARK-8608][SPARK-8609][SQL] reset mutable st...

2015-07-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7674#issuecomment-124976473 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] spark pull request: [SPARK-8608][SPARK-8609][SQL] reset mutable st...

2015-07-26 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7674#discussion_r35495343 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala --- @@ -30,33 +30,28 @@ import org.apache.spark.sql.test.{ExamplePointUDT, ExamplePo

[GitHub] spark pull request: [SPARK-8608][SPARK-8609][SQL] reset mutable st...

2015-07-26 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/7674#discussion_r35495354 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala --- @@ -589,21 +581,17 @@ class DataFrameSuite extends QueryTest with SQLTestUtils {

[GitHub] spark pull request: [SPARK-8608][SPARK-8609][SQL] reset mutable st...

2015-07-26 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/7674#issuecomment-125022048 @cloud-fan I'm confused -- what did this change do? Is it the change in the analyzer that fixes the bug? If yes, can you add a unit test to the analyzer rule, in addition t