[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/9011 --- 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-10960][SQL] SQL with windowing function...

2015-10-12 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/9011#discussion_r41773979 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala --- @@ -833,6 +833,33 @@ class SQLQuerySuite extends QueryTest with

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-12 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-147449167 LGTM. @viirya Thank you for fixing this bug. --- 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-10960][SQL] SQL with windowing function...

2015-10-12 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-147449256 Merging to master and 1.5 branch. --- 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-10960][SQL] SQL with windowing function...

2015-10-12 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/9011#discussion_r41773288 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala --- @@ -833,6 +833,33 @@ class SQLQuerySuite extends QueryTest with

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-11 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-147212401 ping @cloud-fan Any other comments? --- 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] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-11 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-147293397 cc @yhuai to take another look. --- 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] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-09 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/9011#discussion_r41691082 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -831,6 +838,13 @@ class Analyzer( val

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-147020283 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-10960][SQL] SQL with windowing function...

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

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-147020290 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-10960][SQL] SQL with windowing function...

2015-10-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-147028680 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] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

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

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

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

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-09 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-146932670 @cloud-fan Thanks for suggesting. I've updated 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

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-09 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/9011#discussion_r41666551 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala --- @@ -831,6 +838,13 @@ class Analyzer( val

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-08 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-146438339 cc @cloud-fan can you take a look at this? --- 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-10960][SQL] SQL with windowing function...

2015-10-08 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-146677593 Hi @viirya , can you add more description about the reason of this bug? e.g. why does this bug exist and how do you handle it, so that other people can understand

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-146313115 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] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

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

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-146313121 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-07 Thread viirya
GitHub user viirya opened a pull request: https://github.com/apache/spark/pull/9011 [SPARK-10960][SQL] SQL with windowing function should be able to refer column in inner select JIRA: https://issues.apache.org/jira/browse/SPARK-10960 Currently, when accessing a column in

[GitHub] spark pull request: [SPARK-10960][SQL] SQL with windowing function...

2015-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-146283043 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-10960][SQL] SQL with windowing function...

2015-10-07 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9011#issuecomment-146283006 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-10960][SQL] SQL with windowing function...

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