[GitHub] spark issue #14846: [SPARK-17274][SQL] Move join optimizer rules into a sepa...

2016-08-26 Thread yhuai
Github user yhuai commented on the issue: https://github.com/apache/spark/pull/14846 lgtm --- 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

[GitHub] spark issue #14846: [SPARK-17274][SQL] Move join optimizer rules into a sepa...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14846 **[Test build #64519 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64519/consoleFull)** for PR 14846 at commit

[GitHub] spark issue #14846: [SPARK-17274][SQL] Move join optimizer rules into a sepa...

2016-08-26 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/14846 Hopefully this one can be merged in branch-2.0. --- 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] spark pull request #14846: [SPARK-17274][SQL] Move join optimizer rules into...

2016-08-26 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/14846 [SPARK-17274][SQL] Move join optimizer rules into a separate file ## What changes were proposed in this pull request? As part of breaking Optimizer.scala apart, this patch moves various join

[GitHub] spark issue #14845: [SPARK-17273][SQL] Move expression optimizer rules into ...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14845 **[Test build #64518 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64518/consoleFull)** for PR 14845 at commit

[GitHub] spark issue #14845: [SPARK-17273][SQL] Move expression optimizer rules into ...

2016-08-26 Thread yhuai
Github user yhuai commented on the issue: https://github.com/apache/spark/pull/14845 lgtm --- 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

[GitHub] spark issue #14844: [SPARK-17272][SQL] Move subquery optimizer rules into it...

2016-08-26 Thread yhuai
Github user yhuai commented on the issue: https://github.com/apache/spark/pull/14844 lgtm --- 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

[GitHub] spark pull request #14845: [SPARK-17273][SQL] Move expression optimizer rule...

2016-08-26 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/14845 [SPARK-17273][SQL] Move expression optimizer rules into a separate file ## What changes were proposed in this pull request? As part of breaking Optimizer.scala apart, this patch moves various

[GitHub] spark pull request #14746: [SPARK-17180] [SQL] Fix View Resolution Order in ...

2016-08-26 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/14746#discussion_r76510475 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala --- @@ -105,7 +105,13 @@ case class CreateViewCommand( }

[GitHub] spark issue #14841: [SPARK-17271] [SQL] Planner adds un-necessary Sort even ...

2016-08-26 Thread tejasapatil
Github user tejasapatil commented on the issue: https://github.com/apache/spark/pull/14841 cc'ing @rxin and @hvanhovell for review --- 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] spark issue #14844: [SPARK-17272][SQL] Move subquery optimizer rules into it...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14844 **[Test build #64517 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64517/consoleFull)** for PR 14844 at commit

[GitHub] spark pull request #14844: [SPARK-17272][SQL] Move subquery optimizer rules ...

2016-08-26 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/14844 [SPARK-17272][SQL] Move subquery optimizer rules into its own file ## What changes were proposed in this pull request? As part of breaking Optimizer.scala apart, this patch moves various subquery

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14836 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 project does not have this feature

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14836 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64513/ Test FAILed. ---

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14836 **[Test build #64513 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64513/consoleFull)** for PR 14836 at commit

[GitHub] spark issue #14843: [SPARK-17270][SQL] Move object optimization rules into i...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14843 **[Test build #64516 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64516/consoleFull)** for PR 14843 at commit

[GitHub] spark pull request #14843: [SPARK-17270][SQL] Move object optimization rules...

2016-08-26 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/14843 [SPARK-17270][SQL] Move object optimization rules into its own file (branch-2.0) ## What changes were proposed in this pull request? As part of breaking Optimizer.scala apart, this patch moves

[GitHub] spark issue #14841: [SPARK-17271] [SQL] Planner adds un-necessary Sort even ...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14841 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64512/ Test PASSed. ---

[GitHub] spark issue #14841: [SPARK-17271] [SQL] Planner adds un-necessary Sort even ...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14841 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 does not have this feature

[GitHub] spark issue #14841: [SPARK-17271] [SQL] Planner adds un-necessary Sort even ...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14841 **[Test build #64512 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64512/consoleFull)** for PR 14841 at commit

[GitHub] spark issue #14838: [SPARK-17269][SQL] Move finish analysis optimization sta...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14838 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64515/ Test FAILed. ---

[GitHub] spark pull request #14838: [SPARK-17269][SQL] Move finish analysis optimizat...

2016-08-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14838 --- 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 issue #14838: [SPARK-17269][SQL] Move finish analysis optimization sta...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14838 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 project does not have this feature

[GitHub] spark issue #14840: [SPARK-16216][SQL][FOLLOWUP][BRANCH-2.0] Bacoport enabli...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14840 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64511/ Test PASSed. ---

[GitHub] spark issue #14840: [SPARK-16216][SQL][FOLLOWUP][BRANCH-2.0] Bacoport enabli...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14840 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 does not have this feature

[GitHub] spark issue #14840: [SPARK-16216][SQL][FOLLOWUP][BRANCH-2.0] Bacoport enabli...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14840 **[Test build #64511 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64511/consoleFull)** for PR 14840 at commit

[GitHub] spark issue #14839: [SPARK-17270][SQL] Move object optimization rules into i...

2016-08-26 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/14839 OK I will submit a separate one. --- 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

[GitHub] spark issue #14839: [SPARK-17270][SQL] Move object optimization rules into i...

2016-08-26 Thread yhuai
Github user yhuai commented on the issue: https://github.com/apache/spark/pull/14839 Seems the conflicts are caused by https://github.com/apache/spark/commit/d063898bebaaf4ec2aad24c3ac70aabdbf97a190#diff-a636a87d8843eeccca90140be91d4faf. --- If your project is set up for it, you can

[GitHub] spark issue #14839: [SPARK-17270][SQL] Move object optimization rules into i...

2016-08-26 Thread yhuai
Github user yhuai commented on the issue: https://github.com/apache/spark/pull/14839 hmm. There are conflicts in branch 2.0. --- 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 #14839: [SPARK-17270][SQL] Move object optimization rules...

2016-08-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14839 --- 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 issue #14838: [SPARK-17269][SQL] Move finish analysis optimization sta...

2016-08-26 Thread yhuai
Github user yhuai commented on the issue: https://github.com/apache/spark/pull/14838 LGTM. Merging to master and branch 2.0. --- 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 issue #14839: [SPARK-17270][SQL] Move object optimization rules into i...

2016-08-26 Thread yhuai
Github user yhuai commented on the issue: https://github.com/apache/spark/pull/14839 LGTM. Merging to master and branch 2.0. --- 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 issue #14842: [SPARK-10747][SQL] Support NULS FIRST|LAST clause in ORD...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14842 **[Test build #64514 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64514/consoleFull)** for PR 14842 at commit

[GitHub] spark issue #14842: [SPARK-10747][SQL] Support NULS FIRST|LAST clause in ORD...

2016-08-26 Thread xwu0226
Github user xwu0226 commented on the issue: https://github.com/apache/spark/pull/14842 @hvanhovell It turns out that this involves the whole new feature that supports NULLS FIRST|LAST in ORDER BY clause. Please help to see if the implementation is in the right direction. Thanks!

[GitHub] spark pull request #14842: [SPARK-10747][SQL] Support NULS FIRST|LAST clause...

2016-08-26 Thread xwu0226
GitHub user xwu0226 opened a pull request: https://github.com/apache/spark/pull/14842 [SPARK-10747][SQL] Support NULS FIRST|LAST clause in ORDER BY ## What changes were proposed in this pull request? Currently, ORDER BY clause returns nulls value according to sorting order

[GitHub] spark issue #12601: [SPARK-14525][SQL] Make DataFrameWrite.save work for jdb...

2016-08-26 Thread JustinPihony
Github user JustinPihony commented on the issue: https://github.com/apache/spark/pull/12601 @rxin I am sure you are busy, so if there is anybody else I can ping to have this reviewed please let me know. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14836 **[Test build #64513 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64513/consoleFull)** for PR 14836 at commit

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread keypointt
Github user keypointt commented on the issue: https://github.com/apache/spark/pull/14836 Jenkins retest this please --- 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

[GitHub] spark issue #14841: [SPARK-17271] [SQL] Planner adds un-necessary Sort even ...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14841 **[Test build #64512 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64512/consoleFull)** for PR 14841 at commit

[GitHub] spark issue #14841: [SPARK-17271] [SQL] Planner adds un-necessary Sort even ...

2016-08-26 Thread tejasapatil
Github user tejasapatil commented on the issue: https://github.com/apache/spark/pull/14841 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 enabled and wishes so, or

[GitHub] spark pull request #14841: [SPARK-17271] [SQL] Planner adds un-necessary Sor...

2016-08-26 Thread tejasapatil
GitHub user tejasapatil opened a pull request: https://github.com/apache/spark/pull/14841 [SPARK-17271] [SQL] Planner adds un-necessary Sort even if child ordering is semantically same as required ordering ## What changes were proposed in this pull request? Jira :

[GitHub] spark issue #14839: [SPARK-17270][SQL] Move object optimization rules into i...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14839 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64510/ Test PASSed. ---

[GitHub] spark issue #14839: [SPARK-17270][SQL] Move object optimization rules into i...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14839 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 does not have this feature

[GitHub] spark issue #14839: [SPARK-17270][SQL] Move object optimization rules into i...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14839 **[Test build #64510 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64510/consoleFull)** for PR 14839 at commit

[GitHub] spark issue #14838: [SPARK-17269][SQL] Move finish analysis optimization sta...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14838 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64509/ Test PASSed. ---

[GitHub] spark issue #14838: [SPARK-17269][SQL] Move finish analysis optimization sta...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14838 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 does not have this feature

[GitHub] spark issue #14840: [SPARK-16216][SQL][FOLLOWUP][BRANCH-2.0] Bacoport enabli...

2016-08-26 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/14840 I will cc you @hvanhovell . This is the backport of #14829. --- 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 issue #14838: [SPARK-17269][SQL] Move finish analysis optimization sta...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14838 **[Test build #64509 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64509/consoleFull)** for PR 14838 at commit

[GitHub] spark issue #14840: [SPARK-16216][SQL][FOLLOWUP][BRANCH-2.0] Bacoport enabli...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14840 **[Test build #64511 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64511/consoleFull)** for PR 14840 at commit

[GitHub] spark pull request #14840: [SPARK-16216][SQL][FOLLOWUP][BRANCH-2.0] Bacoport...

2016-08-26 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/14840 [SPARK-16216][SQL][FOLLOWUP][BRANCH-2.0] Bacoport enabling timestamp type tests for JSON and verify all unsupported types in CSV ## What changes were proposed in this pull request?

[GitHub] spark issue #14835: [SPARK-17243] [Web UI] Spark 2.0 History Server won't lo...

2016-08-26 Thread QQshu1
Github user QQshu1 commented on the issue: https://github.com/apache/spark/pull/14835 I may have a question ,if I just have a application,and it is very large .It also can't load it by this way. How to solve this kind of scene? --- If your project is set up for it, you can reply

[GitHub] spark pull request #14837: [SPARK-17266] [TEST] Add empty strings to the reg...

2016-08-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14837 --- 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 issue #14837: [SPARK-17266] [TEST] Add empty strings to the regression...

2016-08-26 Thread yhuai
Github user yhuai commented on the issue: https://github.com/apache/spark/pull/14837 Thanks. I will merge this and watch the build. --- 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] spark issue #14835: [SPARK-17243] [Web UI] Spark 2.0 History Server won't lo...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14835 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 does not have this feature

[GitHub] spark issue #14835: [SPARK-17243] [Web UI] Spark 2.0 History Server won't lo...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14835 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64508/ Test PASSed. ---

[GitHub] spark issue #14835: [SPARK-17243] [Web UI] Spark 2.0 History Server won't lo...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14835 **[Test build #64508 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64508/consoleFull)** for PR 14835 at commit

[GitHub] spark issue #14832: [SPARK-17263][SQL] Add hexadecimal literal parsing

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14832 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 does not have this feature

[GitHub] spark issue #14832: [SPARK-17263][SQL] Add hexadecimal literal parsing

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14832 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64507/ Test PASSed. ---

[GitHub] spark issue #14832: [SPARK-17263][SQL] Add hexadecimal literal parsing

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14832 **[Test build #64507 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64507/consoleFull)** for PR 14832 at commit

[GitHub] spark pull request #14746: [SPARK-17180] [SQL] Fix View Resolution Order in ...

2016-08-26 Thread cloud-fan
Github user cloud-fan commented on a diff in the pull request: https://github.com/apache/spark/pull/14746#discussion_r76508097 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala --- @@ -105,7 +105,13 @@ case class CreateViewCommand( }

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14836 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64504/ Test FAILed. ---

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14836 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 project does not have this feature

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14836 **[Test build #64504 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64504/consoleFull)** for PR 14836 at commit

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14836 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 project does not have this feature

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14836 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64506/ Test FAILed. ---

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14836 **[Test build #64506 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64506/consoleFull)** for PR 14836 at commit

[GitHub] spark issue #14837: [SPARK-17266] [TEST] Add empty strings to the regression...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14837 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 does not have this feature

[GitHub] spark issue #14837: [SPARK-17266] [TEST] Add empty strings to the regression...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14837 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64505/ Test PASSed. ---

[GitHub] spark issue #14837: [SPARK-17266] [TEST] Add empty strings to the regression...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14837 **[Test build #64505 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64505/consoleFull)** for PR 14837 at commit

[GitHub] spark issue #14712: [SPARK-17072] [SQL] support table-level statistics gener...

2016-08-26 Thread wzhfy
Github user wzhfy commented on the issue: https://github.com/apache/spark/pull/14712 @cloud-fan did you see the [comment](https://github.com/apache/spark/pull/14712#discussion_r75540560) by @gatorsmile ? --- If your project is set up for it, you can reply to this email and have

[GitHub] spark issue #14839: [SPARK-17270][SQL] Move object optimization rules into i...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14839 **[Test build #64510 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64510/consoleFull)** for PR 14839 at commit

[GitHub] spark pull request #14839: [SPARK-17270][SQL] Move object optimization rules...

2016-08-26 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/14839 [SPARK-17270][SQL] Move object optimization rules into its own file ## What changes were proposed in this pull request? As part of breaking Optimizer.scala apart, this patch moves various Dataset

[GitHub] spark issue #14838: [SPARK-17269][SQL] Move finish analysis optimization sta...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14838 **[Test build #64509 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64509/consoleFull)** for PR 14838 at commit

[GitHub] spark issue #14839: [SPARK-17270][SQL] Move object optimization rules into i...

2016-08-26 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/14839 cc @cloud-fan --- 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

[GitHub] spark pull request #14838: [SPARK-17269][SQL] Move finish analysis optimizat...

2016-08-26 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/14838#discussion_r76506795 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/finishAnalysis.scala --- @@ -0,0 +1,65 @@ +/* + * Licensed to the Apache

[GitHub] spark issue #14838: [SPARK-17269][SQL] Move finish analysis optimization sta...

2016-08-26 Thread rxin
Github user rxin commented on the issue: https://github.com/apache/spark/pull/14838 cc @srinathshankar --- 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

[GitHub] spark pull request #14838: [SPARK-17269][SQL] Move finish analysis optimizat...

2016-08-26 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/14838 [SPARK-17269][SQL] Move finish analysis optimization stage into its own file ## What changes were proposed in this pull request? As part of breaking Optimizer.scala apart, this patch moves various

[GitHub] spark pull request #14691: [SPARK-16407][STREAMING] Allow users to supply cu...

2016-08-26 Thread holdenk
Github user holdenk commented on a diff in the pull request: https://github.com/apache/spark/pull/14691#discussion_r76505266 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/streaming/DataStreamWriter.scala --- @@ -123,12 +124,30 @@ final class DataStreamWriter[T]

[GitHub] spark pull request #14691: [SPARK-16407][STREAMING] Allow users to supply cu...

2016-08-26 Thread frreiss
Github user frreiss commented on a diff in the pull request: https://github.com/apache/spark/pull/14691#discussion_r76505064 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/streaming/DataStreamWriter.scala --- @@ -123,12 +124,30 @@ final class DataStreamWriter[T]

[GitHub] spark pull request #14691: [SPARK-16407][STREAMING] Allow users to supply cu...

2016-08-26 Thread frreiss
Github user frreiss commented on a diff in the pull request: https://github.com/apache/spark/pull/14691#discussion_r76504239 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/streaming/DataStreamWriter.scala --- @@ -123,12 +124,30 @@ final class DataStreamWriter[T]

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread keypointt
Github user keypointt commented on the issue: https://github.com/apache/spark/pull/14836 I just noticed that there are many other places where 'PropSpec' style is used `with Matchers`, while I guess spark project is mainly using `FunSuite` sylte --- If your project is set up for

[GitHub] spark issue #12004: [SPARK-7481][build] [WIP] Add Hadoop 2.7+ spark-cloud mo...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/12004 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64503/ Test PASSed. ---

[GitHub] spark issue #12004: [SPARK-7481][build] [WIP] Add Hadoop 2.7+ spark-cloud mo...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/12004 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 does not have this feature

[GitHub] spark pull request #14773: [SPARK-17203][SQL] data source options should alw...

2016-08-26 Thread frreiss
Github user frreiss commented on a diff in the pull request: https://github.com/apache/spark/pull/14773#discussion_r76503740 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala --- @@ -65,7 +65,7 @@ case class

[GitHub] spark issue #12004: [SPARK-7481][build] [WIP] Add Hadoop 2.7+ spark-cloud mo...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/12004 **[Test build #64503 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64503/consoleFull)** for PR 12004 at commit

[GitHub] spark pull request #14815: [SPARK-17244] Catalyst should not pushdown non-de...

2016-08-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/14815 --- 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 issue #14835: [SPARK-17243] [Web UI] Spark 2.0 History Server won't lo...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14835 **[Test build #64508 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64508/consoleFull)** for PR 14835 at commit

[GitHub] spark issue #14837: [SPARK-17266] [TEST] Add empty strings to the regression...

2016-08-26 Thread JoshRosen
Github user JoshRosen commented on the issue: https://github.com/apache/spark/pull/14837 LGTM --- 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

[GitHub] spark issue #14815: [SPARK-17244] Catalyst should not pushdown non-determini...

2016-08-26 Thread yhuai
Github user yhuai commented on the issue: https://github.com/apache/spark/pull/14815 LGTM. Merging to master and branch 2.0. --- 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 issue #14832: [SPARK-17263][SQL] Add hexadecimal literal parsing

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14832 **[Test build #64507 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64507/consoleFull)** for PR 14832 at commit

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14836 **[Test build #64506 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64506/consoleFull)** for PR 14836 at commit

[GitHub] spark pull request #14836: [MINOR][MLlib][SQL] Clean up unused variables and...

2016-08-26 Thread keypointt
Github user keypointt commented on a diff in the pull request: https://github.com/apache/spark/pull/14836#discussion_r76502969 --- Diff: core/src/test/scala/org/apache/spark/AccumulatorSuite.scala --- @@ -171,7 +171,7 @@ class AccumulatorSuite extends SparkFunSuite with Matchers

[GitHub] spark issue #14835: [SPARK-17243] [Web UI] Spark 2.0 History Server won't lo...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14835 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64502/ Test PASSed. ---

[GitHub] spark issue #14835: [SPARK-17243] [Web UI] Spark 2.0 History Server won't lo...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14835 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 does not have this feature

[GitHub] spark issue #14835: [SPARK-17243] [Web UI] Spark 2.0 History Server won't lo...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14835 **[Test build #64502 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64502/consoleFull)** for PR 14835 at commit

[GitHub] spark issue #14835: [SPARK-17243] [Web UI] Spark 2.0 History Server won't lo...

2016-08-26 Thread ajbozarth
Github user ajbozarth commented on the issue: https://github.com/apache/spark/pull/14835 So I decided to switch to my alternate implementation after thinking hard about how detrimental limiting the api would be. I also updated the docs with my new conf and api param. I will push

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14836 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 does not have this feature

[GitHub] spark pull request #14836: [MINOR][MLlib][SQL] Clean up unused variables and...

2016-08-26 Thread keypointt
Github user keypointt commented on a diff in the pull request: https://github.com/apache/spark/pull/14836#discussion_r76502012 --- Diff: core/src/test/scala/org/apache/spark/AccumulatorSuite.scala --- @@ -100,7 +100,7 @@ class AccumulatorSuite extends SparkFunSuite with Matchers

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/14836 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64501/ Test PASSed. ---

[GitHub] spark issue #14836: [MINOR][MLlib][SQL] Clean up unused variables and unused...

2016-08-26 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/14836 **[Test build #64501 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64501/consoleFull)** for PR 14836 at commit

  1   2   3   4   5   >