[GitHub] spark pull request: [SPARK-9826][Core] Fix cannot use custom class...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8109#issuecomment-130322209 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-9879][SQL] Fix OOM in Limit clause with...

2015-08-12 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/8128#discussion_r36866693 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala --- @@ -224,6 +225,56 @@ case class Limit(limit: Int, child:

[GitHub] spark pull request: [SPARK-9634][SPARK-9323][SQL] cleanup unnecess...

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

[GitHub] spark pull request: [SPARK-9879][SQL][WIP] Fix OOM in Limit clause...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8128#issuecomment-130324459 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9879][SQL][WIP] Fix OOM in Limit clause...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8128#issuecomment-130324416 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9613] [CORE] [WIP] Ban use of JavaConve...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8033#issuecomment-130324603 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-8625] [Core] Propagate user exceptions ...

2015-08-12 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/7014#issuecomment-130336611 merged to master 1.5 thanks @tomwhite ! --- 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-8625] [Core] Propagate user exceptions ...

2015-08-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/7014 --- 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-9634][SPARK-9323][SQL] cleanup unnecess...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7957#issuecomment-130336696 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-9634][SPARK-9323][SQL] cleanup unnecess...

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

[GitHub] spark pull request: [SPARK-9645] [yarn] [core] Allow shuffle servi...

2015-08-12 Thread dragos
Github user dragos commented on the pull request: https://github.com/apache/spark/pull/7966#issuecomment-130337132 Sorry for that. I was following [this line](https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/util/Utils.scala#L774) which doesn't seem to

[GitHub] spark pull request: [SPARK-9826][Core] Fix cannot use custom class...

2015-08-12 Thread michellemay
Github user michellemay commented on the pull request: https://github.com/apache/spark/pull/8109#issuecomment-130340256 @vanzin about your comment on ShutdownHookManager, should I rename it? Also, while refactoring calls to Utils.addShutdownHook, I've seen instances of

[GitHub] spark pull request: [SPARK-9879][SQL][WIP] Fix OOM in Limit clause...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8128#issuecomment-130341196 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-9740] [SPARK-9592] [SQL] Change the def...

2015-08-12 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/8113#discussion_r36873052 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala --- @@ -130,24 +147,67 @@ case class

[GitHub] spark pull request: [SPARK-9740] [SPARK-9592] [SQL] Change the def...

2015-08-12 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/8113#discussion_r36873218 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/functions.scala --- @@ -130,24 +147,67 @@ case class

[GitHub] spark pull request: [SPARK-9634][SPARK-9323][SQL] cleanup unnecess...

2015-08-12 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/7957#issuecomment-130321856 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-9826][Core] Fix cannot use custom class...

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

[GitHub] spark pull request: [SPARK-9613] [CORE] [WIP] Ban use of JavaConve...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8033#issuecomment-130322961 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9879][SQL] Fix OOM in Limit clause with...

2015-08-12 Thread chenghao-intel
GitHub user chenghao-intel opened a pull request: https://github.com/apache/spark/pull/8128 [SPARK-9879][SQL] Fix OOM in Limit clause with large number ``` create table tablsetest as select * from dpa_ord_bill_tf order by member_id limit 2000 ``` It causes OOM,

[GitHub] spark pull request: [SPARK-9634][SPARK-9323][SQL] cleanup unnecess...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7957#issuecomment-130322990 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9613] [CORE] [WIP] Ban use of JavaConve...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8033#issuecomment-130322991 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9634][SPARK-9323][SQL] cleanup unnecess...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7957#issuecomment-130322966 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9879][SQL][WIP] Fix OOM in Limit clause...

2015-08-12 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/8128#discussion_r36867308 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala --- @@ -224,6 +225,56 @@ case class Limit(limit: Int, child:

[GitHub] spark pull request: [SPARK-9879][SQL][WIP] Fix OOM in Limit clause...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8128#issuecomment-130326577 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-9879][SQL][WIP] Fix OOM in Limit clause...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8128#issuecomment-130336313 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9879][SQL] Fix OOM in Limit clause with...

2015-08-12 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/8128#discussion_r36866531 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SQLConf.scala --- @@ -192,6 +192,12 @@ private[spark] object SQLConf { column based

[GitHub] spark pull request: [SPARK-9879][SQL][WIP] Fix OOM in Limit clause...

2015-08-12 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/8128#issuecomment-130324170 cc @JoshRosen @cloud-fan @liancheng --- 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-6190][core] create LargeByteBuffer for ...

2015-08-12 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/5400#issuecomment-130331352 Hey @tgravescs , sorry for the late response. Absolutely, it would be great to get you to start testing this. I'm a bit overloaded at the moment but probably next

[GitHub] spark pull request: [SPARK-9879][SQL][WIP] Fix OOM in Limit clause...

2015-08-12 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/8128#issuecomment-130335280 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

[GitHub] spark pull request: [SPARK-8890][SQL] Fallback on sorting when wri...

2015-08-12 Thread yjshen
Github user yjshen commented on a diff in the pull request: https://github.com/apache/spark/pull/8010#discussion_r36870595 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InsertIntoHadoopFsRelation.scala --- @@ -0,0 +1,165 @@ +/* + * Licensed

[GitHub] spark pull request: [SPARK-9879][SQL][WIP] Fix OOM in Limit clause...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8128#issuecomment-130336342 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9439] [wip] [yarn] External shuffle ser...

2015-08-12 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/7943#discussion_r36873941 --- Diff: network/shuffle/src/main/java/org/apache/spark/network/shuffle/ExternalShuffleBlockResolver.java --- @@ -252,4 +338,118 @@ public String toString()

[GitHub] spark pull request: [SPARK-9740] [SPARK-9592] [SQL] Change the def...

2015-08-12 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/8113#issuecomment-130343660 One more small thing. We should probably also add the ```ignoreNulls``` option to the ```first``` and ```last``` dataframe functions. --- If your project is set up

[GitHub] spark pull request: [SPARK-9556] [SPARK-9619] [SPARK-9624] [STREAM...

2015-08-12 Thread dragos
Github user dragos commented on a diff in the pull request: https://github.com/apache/spark/pull/7913#discussion_r36874133 --- Diff: streaming/src/test/scala/org/apache/spark/streaming/scheduler/ReceiverTrackerSuite.scala --- @@ -67,69 +62,73 @@ class ReceiverTrackerSuite extends

[GitHub] spark pull request: [SPARK-9556] [SPARK-9619] [SPARK-9624] [STREAM...

2015-08-12 Thread dragos
Github user dragos commented on the pull request: https://github.com/apache/spark/pull/7913#issuecomment-130344159 I'm late to the party, I have a few minor comments but other unworthy nitpicks. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [MESOS][SPARK-8873] Clean up external shuffle ...

2015-08-12 Thread dragos
Github user dragos commented on a diff in the pull request: https://github.com/apache/spark/pull/7820#discussion_r36875672 --- Diff: sbin/start-mesos-shuffle-service.sh --- @@ -0,0 +1,35 @@ +#!/usr/bin/env bash + +# +# Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-9199][CORE] Update Tachyon dependency f...

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

[GitHub] spark pull request: [SPARK-9757] [SQL] Fixes persistence of Parque...

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

[GitHub] spark pull request: [SPARK-9580] [SQL] Replace singletons in SQL t...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8111#issuecomment-130451987 [Test build #1503 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1503/consoleFull) for PR 8111 at commit

[GitHub] spark pull request: [SPARK-9580] [SQL] Replace singletons in SQL t...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8111#issuecomment-130452075 [Test build #1505 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1505/consoleFull) for PR 8111 at commit

[GitHub] spark pull request: [SPARK-9580] [SQL] Replace singletons in SQL t...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8111#issuecomment-130452048 [Test build #1504 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1504/consoleFull) for PR 8111 at commit

[GitHub] spark pull request: [SPARK-9580] [SQL] Replace singletons in SQL t...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8111#issuecomment-130451965 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-9903] [MLLIB] skip local processing in ...

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

[GitHub] spark pull request: [SPARK-9877][Core] Fix StandaloneRestServer NP...

2015-08-12 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/8127#issuecomment-130453661 @jerryshao weird, I've been running this and it works for me. How did you reproduce it? --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-9789] [ML] Added logreg threshold param...

2015-08-12 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8079#issuecomment-130453515 @dbtsai Thanks for checking this! I'll go ahead and merge it with master and branch-1.5 --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-9894] [SQL] Json writer should handle M...

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

[GitHub] spark pull request: [SPARK-9913] [MLLIB] LDAUtils should be privat...

2015-08-12 Thread mengxr
GitHub user mengxr opened a pull request: https://github.com/apache/spark/pull/8142 [SPARK-9913] [MLLIB] LDAUtils should be private @feynmanliang You can merge this pull request into a Git repository by running: $ git pull https://github.com/mengxr/spark SPARK-9913

[GitHub] spark pull request: [SPARK-9909][ML][TRIVIAL] move weightCol to sh...

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

[GitHub] spark pull request: [SPARK-9903] [MLLIB] skip local processing in ...

2015-08-12 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/8136#discussion_r36922780 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala --- @@ -282,26 +282,31 @@ object PrefixSpan extends Logging {

[GitHub] spark pull request: [SPARK-9757] [SQL] Fixes persistence of Parque...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8130#issuecomment-130467602 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9914] [ML] define setters explicitly fo...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8143#issuecomment-130467578 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9805] [MLLIB] [PYTHON] [STREAMING] Adde...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8087#issuecomment-130467593 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9805] [MLLIB] [PYTHON] [STREAMING] Adde...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8087#issuecomment-130467967 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-9918] [MLLIB] remove runs from k-means ...

2015-08-12 Thread mengxr
GitHub user mengxr opened a pull request: https://github.com/apache/spark/pull/8148 [SPARK-9918] [MLLIB] remove runs from k-means in the pipeline API This requires some discussion. I'm not sure whether `runs` is a useful parameter. It certainly complicates the implementation. We

[GitHub] spark pull request: [SPARK-9580] [SQL] Replace singletons in SQL t...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8111#issuecomment-130470329 [Test build #1504 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1504/console) for PR 8111 at commit

[GitHub] spark pull request: [SPARK-9918] [MLLIB] remove runs from k-means ...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8148#issuecomment-130471360 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-9199][CORE] Update Tachyon dependency f...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8135#issuecomment-130471550 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-9882][Core] Priority-based scheduling f...

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

[GitHub] spark pull request: [SPARK-9908] [SQL] When spark.sql.tungsten.ena...

2015-08-12 Thread yhuai
GitHub user yhuai opened a pull request: https://github.com/apache/spark/pull/8149 [SPARK-9908] [SQL] When spark.sql.tungsten.enabled is false, broadcast join does not work https://issues.apache.org/jira/browse/SPARK-9908 You can merge this pull request into a Git repository by

[GitHub] spark pull request: [SPARK-7736] [core] [yarn] Make pyspark fail Y...

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

[GitHub] spark pull request: [SPARK-9882][Core] Priority-based scheduling f...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8129#issuecomment-130473135 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-9908] [SQL] When spark.sql.tungsten.ena...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8149#issuecomment-130475457 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-9920] [SQL] The simpleString of Tungste...

2015-08-12 Thread yhuai
GitHub user yhuai opened a pull request: https://github.com/apache/spark/pull/8150 [SPARK-9920] [SQL] The simpleString of TungstenAggregate does not show its output https://issues.apache.org/jira/browse/SPARK-9920 You can merge this pull request into a Git repository by running:

[GitHub] spark pull request: [SPARK-9580] [SQL] Replace singletons in SQL t...

2015-08-12 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/8111#issuecomment-130475536 Reviewed 4 of 4 files at r11, 12 of 12 files at r12, 1 of 1 files at r13. Review status: all files reviewed at latest revision, 3 unresolved

[GitHub] spark pull request: [SPARK-9580] [SQL] Replace singletons in SQL t...

2015-08-12 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/8111#discussion_r36927123 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/test/SharedSQLContext.scala --- @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-9922] [ML] rename StringIndexerReverse ...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8152#issuecomment-130483255 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9894] [SQL] Json writer should handle M...

2015-08-12 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/8137#issuecomment-130483271 LGTM, merging to master and 1.5 --- 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-9922] [ML] rename StringIndexerReverse ...

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

[GitHub] spark pull request: [SPARK-9903] [MLLIB] skip local processing in ...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8136#issuecomment-130488995 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9924] [WEB UI] Don't schedule checkForL...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8153#issuecomment-130488984 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-9903] [MLLIB] skip local processing in ...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8136#issuecomment-130489011 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9924] [WEB UI] Don't schedule checkForL...

2015-08-12 Thread mindprince
GitHub user mindprince opened a pull request: https://github.com/apache/spark/pull/8153 [SPARK-9924] [WEB UI] Don't schedule checkForLogs while some of them are already running. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] spark pull request: [SPARK-9832] [SQL] add a thread-safe lookup fo...

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

[GitHub] spark pull request: [SPARK-9877][Core] Fix StandaloneRestServer NP...

2015-08-12 Thread jerryshao
Github user jerryshao commented on the pull request: https://github.com/apache/spark/pull/8127#issuecomment-130490405 @andrewor14 , I just started a local pseudo standalone cluster in my machine with master and worker in one machine, and then submitted a simple SparkPi application

[GitHub] spark pull request: [SPARK-9832] [SQL] add a thread-safe lookup fo...

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

[GitHub] spark pull request: [SPARK-9580] [SQL] Replace singletons in SQL t...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8111#issuecomment-130490661 [Test build #1518 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1518/consoleFull) for PR 8111 at commit

[GitHub] spark pull request: [SPARK-9199][CORE] Update Tachyon dependency f...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8135#issuecomment-130431170 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9870] Disable driver UI and Master REST...

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

[GitHub] spark pull request: SPARK-9210 corrects aggregate function name in...

2015-08-12 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/7557#issuecomment-130439551 In Spark 1.4, `first` and `last` were not in function registry. Right now, `first_value` and `last_value` are pointing to Hive's `first_value` and `last_value`,

[GitHub] spark pull request: [SPARK-9827] [SQL] fix fd leak in UnsafeRowSer...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8116#issuecomment-130446213 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9312] [ML] Added max confidence factor ...

2015-08-12 Thread badriub
Github user badriub commented on a diff in the pull request: https://github.com/apache/spark/pull/7652#discussion_r36916725 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/OneVsRest.scala --- @@ -127,10 +128,16 @@ final class OneVsRestModel private[ml] (

[GitHub] spark pull request: [SPARK-9894] [SQL] Json writer should handle M...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8137#issuecomment-130457314 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9877][Core] Fix StandaloneRestServer NP...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8127#issuecomment-130457358 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-9894] [SQL] Json writer should handle M...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8137#issuecomment-130457327 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9757] [SQL] Fixes persistence of Parque...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8130#issuecomment-130465332 [Test build #1501 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1501/console) for PR 8130 at commit

[GitHub] spark pull request: [SPARK-9915] [ML] stopWords should use StringA...

2015-08-12 Thread mengxr
GitHub user mengxr opened a pull request: https://github.com/apache/spark/pull/8141 [SPARK-9915] [ML] stopWords should use StringArrayParam @hhbyyh You can merge this pull request into a Git repository by running: $ git pull https://github.com/mengxr/spark SPARK-9915

[GitHub] spark pull request: Testing Jenkins do not merge.

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8139#issuecomment-130464972 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-9913] [MLLIB] LDAUtils should be privat...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8142#issuecomment-130466235 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-9915] [ML] stopWords should use StringA...

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

[GitHub] spark pull request: [SPARK-9907][SQL] Python crc32 is mistakenly c...

2015-08-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8138 --- 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-9833] [yarn] Add options to disable del...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8134#issuecomment-130466459 **[Test build #40654 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40654/console)** for PR 8134 at commit

[GitHub] spark pull request: [SPARK-9914] [ML] define setters explicitly fo...

2015-08-12 Thread mengxr
GitHub user mengxr opened a pull request: https://github.com/apache/spark/pull/8143 [SPARK-9914] [ML] define setters explicitly for Java and use setParam group in RFormula The problem with defining setters in the base class is that it doesn't return the correct type in Java.

[GitHub] spark pull request: [SPARK-9833] [yarn] Add options to disable del...

2015-08-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8134#issuecomment-130466537 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: Testing Jenkins do not merge.

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

[GitHub] spark pull request: [SPARK-9917] [ML] add getMin/getMax and doc fo...

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

[GitHub] spark pull request: [SPARK-9580] [SQL] Replace singletons in SQL t...

2015-08-12 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8111#issuecomment-130468805 [Test build #1505 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1505/console) for PR 8111 at commit

[GitHub] spark pull request: [SPARK-7736] [core] [yarn] Make pyspark fail Y...

2015-08-12 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/7751#issuecomment-130472615 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

[GitHub] spark pull request: [SPARK-9903] [MLLIB] skip local processing in ...

2015-08-12 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/8136#discussion_r36925179 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala --- @@ -282,26 +282,31 @@ object PrefixSpan extends Logging {

[GitHub] spark pull request: [SPARK-9918] [MLLIB] remove runs from k-means ...

2015-08-12 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8148#issuecomment-130472616 LGTM, except that the Scala doc for class KMeans needs to be updated. --- 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-9912] [MLLIB] QRDecomposition should us...

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

  1   2   3   4   5   6   7   8   9   10   >