[GitHub] spark pull request: [SPARK-14251][SQL] Add SQL command for printin...

2016-04-01 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/12099#issuecomment-204652575 Thank you, @rxin , @hvanhovell , @andrewor14 , and @DeepSparkBot . :) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [MINOR][SQL] Fix comments styl and correct sev...

2016-04-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12109 --- 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: [MINOR][SQL] Fix comments styl and correct sev...

2016-04-01 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12109#issuecomment-204652108 The doc changes are actually good. Going to merge this in master. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-14123] [WIP] [SQL] Handle CreateFunctio...

2016-04-01 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/12117#issuecomment-204652079 @viirya I have made a few changes based on your PR. I am removing HiveFunctionRegistry and make HiveSessionCatalog to load Hive's builtin functions if necessary. I still

[GitHub] spark pull request: [SPARK-14334] [SQL] add toLocalIterator for Da...

2016-04-01 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12114#issuecomment-204652020 cc @sameeragarwal 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

[GitHub] spark pull request: [SPARK-14338][SQL] Improve `SimplifyConditiona...

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

[GitHub] spark pull request: [SPARK-14285][SQL] Implement common type-safe ...

2016-04-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12077 --- 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-14251][SQL] Add SQL command for printin...

2016-04-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12099 --- 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-14285][SQL] Implement common type-safe ...

2016-04-01 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12077#issuecomment-204651326 Merging in master. Thanks. I will create tickets for follow-up work. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-14251][SQL] Add SQL command for printin...

2016-04-01 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12099#issuecomment-204650933 Thanks - going to merge this in 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

[GitHub] spark pull request: [Spark-14138][SQL][master] Fix generated Speci...

2016-04-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12108 --- 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-14138][SQL][master] Fix generated Speci...

2016-04-01 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12108#issuecomment-204650290 LGTM, merging into 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-14338][SQL] Improve `SimplifyConditiona...

2016-04-01 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/12122#issuecomment-204649453 It's strange. `org.apache.spark.sql.execution.streaming.state.StateStoreSuite.maintenance` test fails here and at my other PR frequently. --- If your

[GitHub] spark pull request: [SPARK-14251][SQL] Add SQL command for printin...

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

[GitHub] spark pull request: [SPARK-14341] [SQL] Throw exception on unsuppo...

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

[GitHub] spark pull request: [SPARK-14341] [SQL] Throw exception on unsuppo...

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12125#issuecomment-204649208 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-14341] [SQL] Throw exception on unsuppo...

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

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

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

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

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

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

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9565#issuecomment-204648944 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-14338][SQL] Improve `SimplifyConditiona...

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

[GitHub] spark pull request: [SPARK-14338][SQL] Improve `SimplifyConditiona...

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12122#issuecomment-204648796 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-14338][SQL] Improve `SimplifyConditiona...

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

[GitHub] spark pull request: [SPARK-14234] [CORE] Executor crashes for Task...

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12031#issuecomment-204648026 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: [SPARK-13992][Core][PySpark][FollowUp] Update ...

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

[GitHub] spark pull request: [SPARK-13992][Core][PySpark][FollowUp] Update ...

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

[GitHub] spark pull request: [SPARK-13992][Core][PySpark][FollowUp] Update ...

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

[GitHub] spark pull request: [SPARK-14244][SQL] Don't use SizeBasedWindowFu...

2016-04-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12040 --- 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: [MINOR][SQL] Fix comments styl and correct sev...

2016-04-01 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/12109#discussion_r58288525 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVRelation.scala --- @@ -42,12 +42,12 @@ object CSVRelation extends

[GitHub] spark pull request: [SPARK-13992][Core][PySpark][FollowUp] Update ...

2016-04-01 Thread lw-lin
Github user lw-lin commented on the pull request: https://github.com/apache/spark/pull/12126#issuecomment-204645372 hi @JoshRosen could you take a look? 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

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

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

[GitHub] spark pull request: [SPARK-13992][Core][PySpark][FollowUp] Update ...

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

[GitHub] spark pull request: [SPARK-13992][Core][PySpark][FollowUp] Update ...

2016-04-01 Thread lw-lin
GitHub user lw-lin opened a pull request: https://github.com/apache/spark/pull/12126 [SPARK-13992][Core][PySpark][FollowUp] Update storagelevel.py ## What changes were proposed in this pull request? storagelevel.py updates ## How was this patch tested? no

[GitHub] spark pull request: [SPARK-14245] [Web UI] Display the user in the...

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

[GitHub] spark pull request: [SPARK-14308][ML][MLLIB] Remove unused mllib t...

2016-04-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12097 --- 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-14244][SQL] Don't use SizeBasedWindowFu...

2016-04-01 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/12040#issuecomment-204642356 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

[GitHub] spark pull request: [SPARK-14245] [Web UI] Display the user in the...

2016-04-01 Thread ajbozarth
Github user ajbozarth commented on the pull request: https://github.com/apache/spark/pull/12123#issuecomment-204642012 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

[GitHub] spark pull request: [SPARK-14251][SQL] Add SQL command for printin...

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

[GitHub] spark pull request: [SPARK-14251][SQL] Add SQL command for printin...

2016-04-01 Thread dongjoon-hyun
Github user dongjoon-hyun commented on a diff in the pull request: https://github.com/apache/spark/pull/12099#discussion_r58288294 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveExplainSuite.scala --- @@ -101,4 +101,19 @@ class HiveExplainSuite extends

[GitHub] spark pull request: [SPARK-14308][ML][MLLIB] Remove unused mllib t...

2016-04-01 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/12097#issuecomment-204641878 LGTM Thanks for separating these changes out! Merging with master --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [MINOR][SQL] Fix comments styl and correct sev...

2016-04-01 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request: https://github.com/apache/spark/pull/12109#discussion_r58288165 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVRelation.scala --- @@ -42,12 +42,12 @@ object CSVRelation extends

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11989#issuecomment-204641056 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-3724][ML] RandomForest: More options fo...

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

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

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

[GitHub] spark pull request: [SPARK-14251][SQL] Add SQL command for printin...

2016-04-01 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/12099#discussion_r58288094 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveExplainSuite.scala --- @@ -101,4 +101,19 @@ class HiveExplainSuite extends QueryTest

[GitHub] spark pull request: [SPARK-14251][SQL] Add SQL command for printin...

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

[GitHub] spark pull request: [SPARK-14338][SQL] Improve `SimplifyConditiona...

2016-04-01 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/12122#issuecomment-204640788 Rebased and squashed. --- 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-14338][SQL] Improve `SimplifyConditiona...

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

[GitHub] spark pull request: [SPARK-14341] [SQL] Throw exception on unsuppo...

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

[GitHub] spark pull request: [SPARK-14341] [SQL] Throw exception on unsuppo...

2016-04-01 Thread bomeng
GitHub user bomeng opened a pull request: https://github.com/apache/spark/pull/12125 [SPARK-14341] [SQL] Throw exception on unsupported create / drop macro ddl ## What changes were proposed in this pull request? We throw an AnalysisException that looks like this:

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

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

[GitHub] spark pull request: [SPARK-14340][DOC] Add Scala Example and Descr...

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

[GitHub] spark pull request: [SPARK-14340][DOC] Add Scala Example and Descr...

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

[GitHub] spark pull request: [SPARK-14340][DOC] Add Scala Example and Descr...

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11844#issuecomment-204638458 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-13538][ML] Add GaussianMixture to ML

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

[GitHub] spark pull request: [SPARK-13538][ML] Add GaussianMixture to ML

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11419#issuecomment-204638409 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-13538][ML] Add GaussianMixture to ML

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

[GitHub] spark pull request: [SPARK-14290][CORE][Network] avoid significant...

2016-04-01 Thread liyezhang556520
Github user liyezhang556520 commented on a diff in the pull request: https://github.com/apache/spark/pull/12083#discussion_r58287737 --- Diff: common/network-common/src/main/java/org/apache/spark/network/protocol/MessageWithHeader.java --- @@ -44,6 +45,14 @@ private long

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

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

[GitHub] spark pull request: [SPARK-14340][DOC] Add Scala Example and Descr...

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

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

2016-04-01 Thread yongtang
Github user yongtang commented on the pull request: https://github.com/apache/spark/pull/11989#issuecomment-204637339 @sethah Thanks for the review. I just updated the pull request with issues addressed. Let me know if there are any further issues. --- If your project is set up for

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

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

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

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

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11989#issuecomment-204637069 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-13784][ML] Persistence for RandomForest...

2016-04-01 Thread GayathriMurali
Github user GayathriMurali commented on the pull request: https://github.com/apache/spark/pull/12118#issuecomment-204635416 @jkbradley Thanks for this. This looks great and clarifies a lot of things I was trying to do. I had one minor comment, except that it looks fine to me. ---

[GitHub] spark pull request: [SPARK-14290][CORE][Network] avoid significant...

2016-04-01 Thread liyezhang556520
Github user liyezhang556520 commented on a diff in the pull request: https://github.com/apache/spark/pull/12083#discussion_r58287444 --- Diff: common/network-common/src/main/java/org/apache/spark/network/protocol/MessageWithHeader.java --- @@ -44,6 +45,14 @@ private long

[GitHub] spark pull request: [SPARK-14320][SQL] Make ColumnarBatch.Row muta...

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12103#issuecomment-204635320 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-14320][SQL] Make ColumnarBatch.Row muta...

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

[GitHub] spark pull request: [SPARK-14320][SQL] Make ColumnarBatch.Row muta...

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

[GitHub] spark pull request: [SPARK-7861][ML] PySpark OneVsRest

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

[GitHub] spark pull request: [SPARK-7861][ML] PySpark OneVsRest

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12124#issuecomment-204635219 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-7861][ML] PySpark OneVsRest

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

[GitHub] spark pull request: [SPARK-14340][DOC] Add Scala Example and Descr...

2016-04-01 Thread zhengruifeng
Github user zhengruifeng commented on a diff in the pull request: https://github.com/apache/spark/pull/11844#discussion_r58287382 --- Diff: docs/ml-clustering.md --- @@ -104,4 +104,31 @@ Refer to the [Java API docs](api/java/org/apache/spark/ml/clustering/LDA.html) f {%

[GitHub] spark pull request: [Minor][DOC] Add python examples for DCT,MinMa...

2016-04-01 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/12063#issuecomment-204634870 @mengxr Ok, I will open the missing JIRAs --- 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-13538][ML] Add GaussianMixture to ML

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

[GitHub] spark pull request: [SPARK-14290][CORE][Network] avoid significant...

2016-04-01 Thread liyezhang556520
Github user liyezhang556520 commented on a diff in the pull request: https://github.com/apache/spark/pull/12083#discussion_r58287274 --- Diff: common/network-common/src/main/java/org/apache/spark/network/protocol/MessageWithHeader.java --- @@ -44,6 +45,14 @@ private long

[GitHub] spark pull request: [SPARK-13784][ML] Persistence for RandomForest...

2016-04-01 Thread GayathriMurali
Github user GayathriMurali commented on a diff in the pull request: https://github.com/apache/spark/pull/12118#discussion_r58287249 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeModels.scala --- @@ -358,3 +376,100 @@ private[ml] object DecisionTreeModelReadWrite {

[GitHub] spark pull request: [SPARK-7861][ML] PySpark OneVsRest

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

[GitHub] spark pull request: [SPARK-7861][ML] PySpark OneVsRest

2016-04-01 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/12124#issuecomment-204634632 @jkbradley @mengxr One more thing to discuss, shall we use parallel for-loop in fit() of OneVsRest just like its Scala companion? --- If your project is

[GitHub] spark pull request: [SPARK-7861][ML] PySpark OneVsRest

2016-04-01 Thread yinxusen
GitHub user yinxusen opened a pull request: https://github.com/apache/spark/pull/12124 [SPARK-7861][ML] PySpark OneVsRest ## What changes were proposed in this pull request? https://issues.apache.org/jira/browse/SPARK-7861 Add PySpark OneVsRest. I implement it with

[GitHub] spark pull request: [SPARK-8402][MLLIB] DP Means Clustering

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

[GitHub] spark pull request: [SPARK-8402][MLLIB] DP Means Clustering

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6880#issuecomment-204633940 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-8402][MLLIB] DP Means Clustering

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

[GitHub] spark pull request: [SPARK-8402][MLLIB] DP Means Clustering

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

[GitHub] spark pull request: [SPARK-14290][CORE][Network] avoid significant...

2016-04-01 Thread liyezhang556520
Github user liyezhang556520 commented on the pull request: https://github.com/apache/spark/pull/12083#issuecomment-204632236 >So if we write a 1M buffer, it can only write NIO_BUFFER_LIMIT (512K). And we need to write the reset 512K again. So in this case, we need to copy 1M + 512K

[GitHub] spark pull request: [SPARK-3724][ML] RandomForest: More options fo...

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

[GitHub] spark pull request: [SPARK-14163][CORE] SumEvaluator and countAppr...

2016-04-01 Thread yongtang
Github user yongtang commented on the pull request: https://github.com/apache/spark/pull/11981#issuecomment-204632233 @srowen Sure let me close this PR. --- 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-14123] [WIP] [SQL] Handle CreateFunctio...

2016-04-01 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/12117#discussion_r58287031 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveSessionCatalog.scala --- @@ -112,4 +124,121 @@ class HiveSessionCatalog(

[GitHub] spark pull request: [SPARK-14163][CORE] SumEvaluator and countAppr...

2016-04-01 Thread yongtang
Github user yongtang closed the pull request at: https://github.com/apache/spark/pull/11981 --- 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-11938][ML] Expose numFeatures in all ML...

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

[GitHub] spark pull request: [SPARK-14123] [WIP] [SQL] Handle CreateFunctio...

2016-04-01 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/12117#discussion_r58287012 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveSessionCatalog.scala --- @@ -112,4 +124,121 @@ class HiveSessionCatalog(

[GitHub] spark pull request: [SPARK-11938][ML] Expose numFeatures in all ML...

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/9936#issuecomment-204632181 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-11938][ML] Expose numFeatures in all ML...

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

[GitHub] spark pull request: [SPARK-14123] [WIP] [SQL] Handle CreateFunctio...

2016-04-01 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/12117#discussion_r58286920 --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala --- @@ -211,10 +241,11 @@ class SQLQuerySuite extends QueryTest

[GitHub] spark pull request: [SPARK-11938][ML] Expose numFeatures in all ML...

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

[GitHub] spark pull request: [SPARK-14123] [WIP] [SQL] Handle CreateFunctio...

2016-04-01 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/12117#discussion_r58286788 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/HiveSqlParser.scala --- @@ -272,10 +272,7 @@ class HiveSqlAstBuilder extends

[GitHub] spark pull request: [SPARK-14338][SQL] Improve `SimplifyConditiona...

2016-04-01 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12122#issuecomment-204630296 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-14123] [WIP] [SQL] Handle CreateFunctio...

2016-04-01 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/12117#discussion_r58286763 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/FunctionRegistry.scala --- @@ -67,9 +70,14 @@ class SimpleFunctionRegistry

[GitHub] spark pull request: [SPARK-14338][SQL] Improve `SimplifyConditiona...

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

[GitHub] spark pull request: [SPARK-14338][SQL] Improve `SimplifyConditiona...

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

  1   2   3   4   5   6   7   8   >