[GitHub] spark pull request: Branch 1.6

2016-04-10 Thread rezacsedu
Github user rezacsedu closed the pull request at: https://github.com/apache/spark/pull/12291 --- 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: Branch 1.6

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12291#issuecomment-208102178 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] spark pull request: Branch 1.6

2016-04-10 Thread rezacsedu
GitHub user rezacsedu opened a pull request: https://github.com/apache/spark/pull/12291 Branch 1.6 ## What changes were proposed in this pull request? (Please fill in changes proposed in this fix) ## How was this patch tested? (Please explain how this

[GitHub] spark pull request: [SPARK-14051][SQL] Implement `Double.NaN==Floa...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11868#issuecomment-208101127 **[Test build #55494 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55494/consoleFull)** for PR 11868 at commit

[GitHub] spark pull request: [MINOR][DOCS] Fix wrong data types in JSON Dat...

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

[GitHub] spark pull request: [MINOR][DOCS] Fix wrong data types in JSON Dat...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12290#issuecomment-208100649 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: [MINOR][DOCS] Fix wrong data types in JSON Dat...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12290#issuecomment-208100626 **[Test build #55493 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55493/consoleFull)** for PR 12290 at commit

[GitHub] spark pull request: [MINOR][DOCS] Fix wrong data types in JSON Dat...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12290#issuecomment-208100143 **[Test build #55493 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55493/consoleFull)** for PR 12290 at commit

[GitHub] spark pull request: [MINOR][DOCS] Fix wrong data types in JSON Dat...

2016-04-10 Thread dongjoon-hyun
GitHub user dongjoon-hyun opened a pull request: https://github.com/apache/spark/pull/12290 [MINOR][DOCS] Fix wrong data types in JSON Datasets example. ## What changes were proposed in this pull request? This PR fixes the `age` data types from `integer` to `long` in `SQL

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208094099 **[Test build #55492 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55492/consoleFull)** for PR 12081 at commit

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208093980 The query is cached??? ``` select current_database() ``` It only evaluates `current_database()` once after we create a new session. Let me clear the

[GitHub] spark pull request: [SPARK-14419] [MINOR] coding style cleanup

2016-04-10 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12289#issuecomment-208093454 cc @jaceklaskowski --- 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: [SPARK-14419] [MINOR] coding style cleanup

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12289#issuecomment-208093624 **[Test build #55491 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55491/consoleFull)** for PR 12289 at commit

[GitHub] spark pull request: [SPARK-14419] [MINOR] coding style cleanup

2016-04-10 Thread davies
GitHub user davies opened a pull request: https://github.com/apache/spark/pull/12289 [SPARK-14419] [MINOR] coding style cleanup ## What changes were proposed in this pull request? Making them more consistent. ## How was this patch tested? Existing tests.

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208085556 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] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208085558 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208085507 **[Test build #55490 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55490/consoleFull)** for PR 12081 at commit

[GitHub] spark pull request: [SPARK-14483][WEBUI] Display user name for eac...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12257#issuecomment-208082616 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-14483][WEBUI] Display user name for eac...

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

[GitHub] spark pull request: [SPARK-14483][WEBUI] Display user name for eac...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12257#issuecomment-208082548 **[Test build #55488 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55488/consoleFull)** for PR 12257 at commit

[GitHub] spark pull request: [SPARK-14362][SPARK-14406][SQL][Follow-up] DDL...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12284#issuecomment-208081163 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] spark pull request: [SPARK-14362][SPARK-14406][SQL][Follow-up] DDL...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12284#issuecomment-208081165 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14362][SPARK-14406][SQL][Follow-up] DDL...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12284#issuecomment-208080895 **[Test build #55489 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55489/consoleFull)** for PR 12284 at commit

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208074946 **[Test build #55490 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55490/consoleFull)** for PR 12081 at commit

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59141311 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -218,14 +219,35 @@ case class CatalogTable(

[GitHub] spark pull request: [SPARK-13432][SQL] add the source file name an...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11301#issuecomment-208064474 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-13432][SQL] add the source file name an...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11301#issuecomment-208064473 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] spark pull request: [SPARK-13432][SQL] add the source file name an...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11301#issuecomment-208064429 **[Test build #55486 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55486/consoleFull)** for PR 11301 at commit

[GitHub] spark pull request: [SPARK-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12288#issuecomment-208062810 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-14518] [SQL] Support Comment in CREATE ...

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

[GitHub] spark pull request: [SPARK-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12288#issuecomment-208062268 **[Test build #55485 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55485/consoleFull)** for PR 12288 at commit

[GitHub] spark pull request: [SPARK-14362][SPARK-14406][SQL][Follow-up] DDL...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12284#issuecomment-208058417 **[Test build #55489 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55489/consoleFull)** for PR 12284 at commit

[GitHub] spark pull request: [SPARK-14483][WEBUI] Display user name for eac...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12257#issuecomment-208057856 **[Test build #55488 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55488/consoleFull)** for PR 12257 at commit

[GitHub] spark pull request: [SPARK-14483][WEBUI] Display user name for eac...

2016-04-10 Thread sarutak
Github user sarutak commented on the pull request: https://github.com/apache/spark/pull/12257#issuecomment-208057447 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

[GitHub] spark pull request: [SPARK-14483][WEBUI] Display user name for eac...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12257#issuecomment-208057365 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] spark pull request: [SPARK-14483][WEBUI] Display user name for eac...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12257#issuecomment-208057366 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14483][WEBUI] Display user name for eac...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12257#issuecomment-208057120 **[Test build #55484 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55484/consoleFull)** for PR 12257 at commit

[GitHub] spark pull request: [SPARK-14098][SQL][WIP] Generate Java code tha...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-208053415 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14098][SQL][WIP] Generate Java code tha...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-208053324 **[Test build #55487 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55487/consoleFull)** for PR 11956 at commit

[GitHub] spark pull request: [SPARK-14098][SQL][WIP] Generate Java code tha...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-208053409 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] spark pull request: [SPARK-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12288#issuecomment-208050822 Sure, no problem. : ) --- 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

[GitHub] spark pull request: [SPARK-14098][SQL][WIP] Generate Java code tha...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-208048336 **[Test build #55487 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55487/consoleFull)** for PR 11956 at commit

[GitHub] spark pull request: [SPARK-13432][SQL] add the source file name an...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11301#issuecomment-208047772 **[Test build #55486 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55486/consoleFull)** for PR 11301 at commit

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59139248 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -218,14 +219,35 @@ case class CatalogTable(

[GitHub] spark pull request: [SPARK-14508][BUILD] Add a new ScalaStyle Rule...

2016-04-10 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/12280#issuecomment-208045946 Hi, @srowen . Could you merge this when you have some time? :) --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/12288#issuecomment-208045739 @gatorsmile Thank you for working on it. How about we hold off the change until we merge https://github.com/apache/spark/pull/12271/files? That PR is adding the field

[GitHub] spark pull request: [SPARK-14415][SQL] All functions should show u...

2016-04-10 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/12185#issuecomment-208045636 Thank you, @yhuai and @rxin . --- 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-14506] [SQL] HiveClientImpl's toHiveTab...

2016-04-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/12275#discussion_r59139167 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala --- @@ -659,9 +659,18 @@ private[hive] class HiveClientImpl(

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208044576 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208044571 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] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208044341 **[Test build #55483 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55483/consoleFull)** for PR 12081 at commit

[GitHub] spark pull request: [SPARK-14362][SPARK-14406][SQL][Follow-up] DDL...

2016-04-10 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/12284#discussion_r59139028 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala --- @@ -191,34 +191,40 @@ case class DropTable( override

[GitHub] spark pull request: [SPARK-14415][SQL] All functions should show u...

2016-04-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12185 --- 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-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12288#issuecomment-208042539 **[Test build #55485 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55485/consoleFull)** for PR 12288 at commit

[GitHub] spark pull request: [SPARK-14415][SQL] All functions should show u...

2016-04-10 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/12185#issuecomment-208042408 Thanks. Merging to 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 not have

[GitHub] spark pull request: [SPARK-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/12288#discussion_r59138825 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveSqlParser.scala --- @@ -257,18 +252,20 @@ class HiveSqlAstBuilder extends

[GitHub] spark pull request: [SPARK-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/12288#discussion_r59138729 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveSqlParser.scala --- @@ -257,18 +252,20 @@ class HiveSqlAstBuilder extends

[GitHub] spark pull request: [SPARK-14362][SPARK-14406][SQL][Follow-up] DDL...

2016-04-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/12284#discussion_r59138655 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/ddl.scala --- @@ -191,34 +191,40 @@ case class DropTable( override def

[GitHub] spark pull request: [SPARK-14483][WEBUI] Display user name for eac...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12257#issuecomment-208036744 **[Test build #55484 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55484/consoleFull)** for PR 12257 at commit

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59138476 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -218,14 +219,35 @@ case class CatalogTable(

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208027013 **[Test build #55483 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55483/consoleFull)** for PR 12081 at commit

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59137856 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveSqlParser.scala --- @@ -134,82 +136,117 @@ class HiveSqlAstBuilder extends

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59137836 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveSqlParser.scala --- @@ -397,11 +429,28 @@ class HiveSqlAstBuilder extends

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59137814 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveSqlParser.scala --- @@ -134,82 +136,117 @@ class HiveSqlAstBuilder extends

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208022714 **[Test build #55482 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55482/consoleFull)** for PR 12081 at commit

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208022724 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] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208022731 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-208022147 **[Test build #55482 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55482/consoleFull)** for PR 12081 at commit

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread xwu0226
Github user xwu0226 commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59137396 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala --- @@ -218,14 +219,35 @@ case class CatalogTable(

[GitHub] spark pull request: [SPARK-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12288#issuecomment-208015254 cc @yhuai @andrewor14 Thanks! --- 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-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12288#issuecomment-208014954 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-14518] [SQL] Support Comment in CREATE ...

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

[GitHub] spark pull request: [SPARK-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12288#issuecomment-208014895 **[Test build #55481 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55481/consoleFull)** for PR 12288 at commit

[GitHub] spark pull request: [SPARK-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12288#issuecomment-208005889 **[Test build #55481 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55481/consoleFull)** for PR 12288 at commit

[GitHub] spark pull request: [SPARK-14481] [SQL] Issue Exceptions for All U...

2016-04-10 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/12255#discussion_r59136088 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala --- @@ -1475,7 +1475,6 @@ class SQLQuerySuite extends

[GitHub] spark pull request: [SPARK-14518] [SQL] Support Comment in CREATE ...

2016-04-10 Thread gatorsmile
GitHub user gatorsmile opened a pull request: https://github.com/apache/spark/pull/12288 [SPARK-14518] [SQL] Support Comment in CREATE VIEW What changes were proposed in this pull request? **HQL Syntax**: [Create

[GitHub] spark pull request: Update KMeansExample.scala

2016-04-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12277 --- 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-14321. [SQL] Reduce date format cost and...

2016-04-10 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12105#issuecomment-208000318 Ping @rajeshbalamohan --- 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

[GitHub] spark pull request: [SPARK-14370][MLLIB]removed duplicate generati...

2016-04-10 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/12176#discussion_r59135877 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/clustering/LDAOptimizer.scala --- @@ -536,13 +532,16 @@ private[clustering] object OnlineLDAOptimizer {

[GitHub] spark pull request: [SPARK-14357] [CORE] Properly handle the root ...

2016-04-10 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/12228#issuecomment-207999135 SGTM; I checked for other instances of this pattern and I don't see any, which is I suppose good news (didn't miss any) and bad (unfortunately don't get to simply

[GitHub] spark pull request: [SPARK-14362][SPARK-14406][SQL][Follow-up] DDL...

2016-04-10 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12284#issuecomment-207994961 cc @yhuai @andrewor14 Thanks! --- 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-14479] [ML] GLM supports output link pr...

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

[GitHub] spark pull request: [SPARK-14479] [ML] GLM supports output link pr...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12287#issuecomment-207991894 **[Test build #55480 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55480/consoleFull)** for PR 12287 at commit

[GitHub] spark pull request: [SPARK-14479] [ML] GLM supports output link pr...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12287#issuecomment-207992035 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-14479] [ML] GLM supports output link pr...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12287#issuecomment-207985899 **[Test build #55480 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55480/consoleFull)** for PR 12287 at commit

[GitHub] spark pull request: [SPARK-14479] [ML] GLM supports output link pr...

2016-04-10 Thread yanboliang
GitHub user yanboliang opened a pull request: https://github.com/apache/spark/pull/12287 [SPARK-14479] [ML] GLM supports output link prediction ## What changes were proposed in this pull request? GLM supports output link prediction. ## How was this patch tested? unit

[GitHub] spark pull request: [SPARK-11593][SQL] Replace catalyst converter ...

2016-04-10 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/9565#discussion_r59133802 --- Diff: mllib/src/main/scala/org/apache/spark/ml/Transformer.scala --- @@ -90,7 +90,7 @@ abstract class UnaryTransformer[IN, OUT, T <: UnaryTransformer[IN,

[GitHub] spark pull request: [SPARK-11593][SQL] Replace catalyst converter ...

2016-04-10 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/9565#discussion_r59133768 --- Diff: mllib/src/main/scala/org/apache/spark/ml/Transformer.scala --- @@ -90,7 +90,7 @@ abstract class UnaryTransformer[IN, OUT, T <: UnaryTransformer[IN,

[GitHub] spark pull request: [SPARK-14510][MLLIB] Add args-checking for LDA...

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

[GitHub] spark pull request: [SPARK-14510][MLLIB] Add args-checking for LDA...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12062#issuecomment-207985476 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-14510][MLLIB] Add args-checking for LDA...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12062#issuecomment-207985444 **[Test build #55479 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55479/consoleFull)** for PR 12062 at commit

[GitHub] spark pull request: [SPARK-14510][MLLIB] Add args-checking for LDA...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12062#issuecomment-207980139 **[Test build #55479 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55479/consoleFull)** for PR 12062 at commit

[GitHub] spark pull request: [SPARK-14508][BUILD] Add a new ScalaStyle Rule...

2016-04-10 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12280#issuecomment-207979770 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-14508][BUILD] Add a new ScalaStyle Rule...

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

[GitHub] spark pull request: [SPARK-14508][BUILD] Add a new ScalaStyle Rule...

2016-04-10 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/12280#issuecomment-207979720 **[Test build #55478 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55478/consoleFull)** for PR 12280 at commit

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59132734 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveSqlParser.scala --- @@ -134,82 +136,117 @@ class HiveSqlAstBuilder extends

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59132688 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveSqlParser.scala --- @@ -134,82 +136,117 @@ class HiveSqlAstBuilder extends

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59132631 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveSqlParser.scala --- @@ -397,11 +429,28 @@ class HiveSqlAstBuilder extends

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59132476 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveSqlParser.scala --- @@ -134,82 +136,117 @@ class HiveSqlAstBuilder extends

[GitHub] spark pull request: [SPARK-14388][SQL][WIP] Implement CREATE TABLE

2016-04-10 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/12271#discussion_r59132489 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveSqlParser.scala --- @@ -134,82 +136,117 @@ class HiveSqlAstBuilder extends

<    1   2   3   >