[GitHub] spark pull request: [SPARK-15165][SQL] Codegen can break because t...

2016-05-17 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/12939#discussion_r63562738 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/package.scala --- @@ -162,7 +162,18 @@ package object util { def

[GitHub] spark pull request: [SPARK-15165][SQL] Codegen can break because t...

2016-05-17 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12939#issuecomment-219785529 LGTM, Merging this into master and 2.0, 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

[GitHub] spark pull request: [SPARK-14615][ML] Use the new ML Vector and Ma...

2016-05-17 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/12627#discussion_r63562728 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/api/python/PythonMLLibAPI.scala --- @@ -1207,15 +1209,18 @@ private[python] class PythonMLLibAPI

[GitHub] spark pull request: [SPARK-15165][SQL] Codegen can break because t...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12939 --- 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-14615][ML] Use the new ML Vector and Ma...

2016-05-17 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/12627#discussion_r63563251 --- Diff: mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala --- @@ -967,4 +968,122 @@ object LogisticRegressionSuite {

[GitHub] spark pull request: [SPARK-15273] YarnSparkHadoopUtil#getOutOfMemo...

2016-05-17 Thread tedyu
Github user tedyu commented on a diff in the pull request: https://github.com/apache/spark/pull/13057#discussion_r63563275 --- Diff: launcher/src/main/java/org/apache/spark/launcher/CommandBuilderUtils.java --- @@ -334,6 +334,18 @@ static void addPermGenSizeOpt(List cmd) {

[GitHub] spark pull request: [SPARK-15165][SQL] Codegen can break because t...

2016-05-17 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12939#issuecomment-219786749 @sarutak Could you send another PR for 1.6 branch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-14346][SQL][follow-up] add tests for CR...

2016-05-17 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/13144#issuecomment-219787089 LGTM. Merging to master and branch 2.0. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-14615][ML] Use the new ML Vector and Ma...

2016-05-17 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/12627#discussion_r63563726 --- Diff: mllib/src/test/scala/org/apache/spark/ml/classification/NaiveBayesSuite.scala --- @@ -206,4 +210,48 @@ object NaiveBayesSuite {

[GitHub] spark pull request: [SPARK-14346][SQL][follow-up] add tests for CR...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13144 --- 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-14615][ML] Use the new ML Vector and Ma...

2016-05-17 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/12627#discussion_r63564073 --- Diff: python/pyspark/ml/feature.py --- @@ -59,6 +60,36 @@ 'Word2Vec', 'Word2VecModel'] +class LabeledPoint(object):

[GitHub] spark pull request: [SPARK-13081][PYSPARK][SPARK_SUBMIT]. Allow se...

2016-05-17 Thread BryanCutler
Github user BryanCutler commented on a diff in the pull request: https://github.com/apache/spark/pull/13146#discussion_r63565228 --- Diff: core/src/main/scala/org/apache/spark/SparkConf.scala --- @@ -62,7 +62,8 @@ class SparkConf(loadDefaults: Boolean) extends Cloneable with

[GitHub] spark pull request: [SPARK-13081][PYSPARK][SPARK_SUBMIT]. Allow se...

2016-05-17 Thread BryanCutler
Github user BryanCutler commented on a diff in the pull request: https://github.com/apache/spark/pull/13146#discussion_r63565781 --- Diff: core/src/test/scala/org/apache/spark/SparkConfSuite.scala --- @@ -281,6 +285,7 @@ class SparkConfSuite extends SparkFunSuite with

[GitHub] spark pull request: [SPARK-14615][ML] Use the new ML Vector and Ma...

2016-05-17 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/12627#discussion_r63566248 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/api/python/PythonMLLibAPI.scala --- @@ -1207,15 +1209,18 @@ private[python] class PythonMLLibAPI

[GitHub] spark pull request: [SPARK-13081][PYSPARK][SPARK_SUBMIT]. Allow se...

2016-05-17 Thread BryanCutler
Github user BryanCutler commented on the pull request: https://github.com/apache/spark/pull/13146#issuecomment-219792598 @zjffdu I made some comments, however I sort of feel that the Python executable is an environment setting and not really a Spark property, so it would be better to

[GitHub] spark pull request: [SPARK-15273] YarnSparkHadoopUtil#getOutOfMemo...

2016-05-17 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/13057#discussion_r63567123 --- Diff: launcher/src/main/java/org/apache/spark/launcher/CommandBuilderUtils.java --- @@ -334,6 +334,18 @@ static void addPermGenSizeOpt(List cmd) {

[GitHub] spark pull request: [SPARK-15365] [SQL]: When table size statistic...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13150#issuecomment-219793393 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-15365] [SQL]: When table size statistic...

2016-05-17 Thread Parth-Brahmbhatt
GitHub user Parth-Brahmbhatt opened a pull request: https://github.com/apache/spark/pull/13150 [SPARK-15365] [SQL]: When table size statistics are not available from metastore, we should fallback to HDFS ## What changes were proposed in this pull request? Currently if a table

[GitHub] spark pull request: [SPARK-6320][SQL] Move planLater method into G...

2016-05-17 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/13147#discussion_r63567867 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/ExperimentalMethods.scala --- @@ -42,7 +43,7 @@ class ExperimentalMethods private[sql]() { *

[GitHub] spark pull request: [SPARK-6320][SQL] Move planLater method into G...

2016-05-17 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/13147#discussion_r63567938 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/QueryPlanner.scala --- @@ -27,6 +27,16 @@ import

[GitHub] spark pull request: [SPARK-10372] [CORE] basic test framework for ...

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

[GitHub] spark pull request: [SPARK-10372] [CORE] basic test framework for ...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8559#issuecomment-219795817 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-10372] [CORE] basic test framework for ...

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

[GitHub] spark pull request: [SPARK-15273] YarnSparkHadoopUtil#getOutOfMemo...

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

[GitHub] spark pull request: [SPARK-14615][ML] Use the new ML Vector and Ma...

2016-05-17 Thread dbtsai
Github user dbtsai commented on a diff in the pull request: https://github.com/apache/spark/pull/12627#discussion_r63569421 --- Diff: mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala --- @@ -967,4 +968,122 @@ object LogisticRegressionSuite {

[GitHub] spark pull request: [SPARK-15263][Core] Make shuffle service dir c...

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

[GitHub] spark pull request: [SPARK-10216][SQL] Avoid creating empty files ...

2016-05-17 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/12855#issuecomment-219797464 Thanks, merging to master and 2.0. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] spark pull request: [SPARK-15263][Core] Make shuffle service dir c...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13042#issuecomment-219797991 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-14346][SQL] Native SHOW CREATE TABLE fo...

2016-05-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/13079#discussion_r63570204 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala --- @@ -626,40 +626,149 @@ case class ShowCreateTableCommand(table:

[GitHub] spark pull request: [SPARK-15263][Core] Make shuffle service dir c...

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

[GitHub] spark pull request: [SPARK-15273] YarnSparkHadoopUtil#getOutOfMemo...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13057#issuecomment-219798061 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-15273] YarnSparkHadoopUtil#getOutOfMemo...

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

[GitHub] spark pull request: [SPARK-15273] YarnSparkHadoopUtil#getOutOfMemo...

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

[GitHub] spark pull request: [SPARK-14615][ML] Use the new ML Vector and Ma...

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

[GitHub] spark pull request: [SPARK-15273] YarnSparkHadoopUtil#getOutOfMemo...

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

[GitHub] spark pull request: [SPARK-15273] YarnSparkHadoopUtil#getOutOfMemo...

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

[GitHub] spark pull request: [SPARK-15296][MLlib] Refactor All Java Tests t...

2016-05-17 Thread techaddict
Github user techaddict commented on the pull request: https://github.com/apache/spark/pull/13101#issuecomment-219801934 ping @andrewor14 @mengxr --- 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-15214][SQL] Code-generation for Generat...

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

[GitHub] spark pull request: [SPARK-15273] YarnSparkHadoopUtil#getOutOfMemo...

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

[GitHub] spark pull request: [SPARK-15273] YarnSparkHadoopUtil#getOutOfMemo...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13057#issuecomment-219802293 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-15214][SQL] Code-generation for Generat...

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

[GitHub] spark pull request: [SPARK-15214][SQL] Code-generation for Generat...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13065#issuecomment-219802386 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-15112][SQL] Allows query plan schema an...

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

[GitHub] spark pull request: [SPARK-15112][SQL] Allows query plan schema an...

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

[GitHub] spark pull request: [SPARK-15112][SQL] Allows query plan schema an...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12952#issuecomment-219803506 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-15273] YarnSparkHadoopUtil#getOutOfMemo...

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

[GitHub] spark pull request: [SPARK-15273] YarnSparkHadoopUtil#getOutOfMemo...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13057#issuecomment-219806015 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-15273] YarnSparkHadoopUtil#getOutOfMemo...

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

[GitHub] spark pull request: [SPARK-15357] Cooperative spilling should chec...

2016-05-17 Thread davies
GitHub user davies opened a pull request: https://github.com/apache/spark/pull/13151 [SPARK-15357] Cooperative spilling should check consumer memory mode ## What changes were proposed in this pull request? Since we support forced spilling for Spillable, which only works in

[GitHub] spark pull request: [SPARK-15357] Cooperative spilling should chec...

2016-05-17 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/13151#issuecomment-219807401 cc @andrewor14 --- 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-10216][SQL]Avoid creating empty files d...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8411 --- 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-10216][SQL] Avoid creating empty files ...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12855 --- 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-15307] [SQL] speed up listing files for...

2016-05-17 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/13094#issuecomment-219807927 @rxin updated. --- 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-15357] Cooperative spilling should chec...

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

[GitHub] spark pull request: [SPARK-15300] Fix writer lock conflict when re...

2016-05-17 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/13082#issuecomment-219808144 @andrewor14 Could you review this since @JoshRosen is on vocation. --- 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-15307] [SQL] speed up listing files for...

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

[GitHub] spark pull request: [SPARK-15353] [CORE] Making peer selection for...

2016-05-17 Thread shubhamchopra
GitHub user shubhamchopra opened a pull request: https://github.com/apache/spark/pull/13152 [SPARK-15353] [CORE] Making peer selection for block replication pluggable ## What changes were proposed in this pull request? This PR makes block replication strategies pluggable.

[GitHub] spark pull request: [SPARK-15353] [CORE] Making peer selection for...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13152#issuecomment-219809650 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-10372] [CORE] basic test framework for ...

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

[GitHub] spark pull request: [SPARK-15031][EXAMPLES][FOLLOW-UP] Make Python...

2016-05-17 Thread MLnick
Github user MLnick commented on the pull request: https://github.com/apache/spark/pull/13135#issuecomment-219809983 This looks ok - though the Scala example doesn't throw the exception - why is that (since it is also setting `thresholds`)? --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-15317][Core]Don't store accumulators fo...

2016-05-17 Thread zsxwing
GitHub user zsxwing opened a pull request: https://github.com/apache/spark/pull/13153 [SPARK-15317][Core]Don't store accumulators for every task in listeners ## What changes were proposed in this pull request? In general, the Web UI doesn't need to store the

[GitHub] spark pull request: [SPARK-15357] Cooperative spilling should chec...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13151#issuecomment-219810982 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-15357] Cooperative spilling should chec...

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

[GitHub] spark pull request: [SPARK-15357] Cooperative spilling should chec...

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

[GitHub] spark pull request: [SPARK-15317][Core]Don't store accumulators fo...

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

[GitHub] spark pull request: [SPARK-15357] Cooperative spilling should chec...

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

[GitHub] spark pull request: [SPARK-15182] [ML] Copy MLlib doc to ML: ml.fe...

2016-05-17 Thread MLnick
Github user MLnick commented on the pull request: https://github.com/apache/spark/pull/12957#issuecomment-219814882 @hhbyyh ok sounds good. LGTM, merged to master/branch-2.0. 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-15182] [ML] Copy MLlib doc to ML: ml.fe...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12957 --- 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][DOCS] Replace remaining 'sqlContext' i...

2016-05-17 Thread MLnick
Github user MLnick commented on the pull request: https://github.com/apache/spark/pull/13125#issuecomment-219816541 Merged to master/branch-2.0. 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

[GitHub] spark pull request: [MINOR][DOCS] Replace remaining 'sqlContext' i...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13125 --- 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-8674] [MLlib] Implementation of a 2 sam...

2016-05-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7075#issuecomment-219818786 [Test build #58715 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58715/consoleFull) for PR 7075 at commit

[GitHub] spark pull request: [MINOR][DOCS] Replace remaining 'sqlContext' i...

2016-05-17 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/13125#issuecomment-219820760 Thank you, @MLnick ! --- 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-15323] Fix reading of partitioned forma...

2016-05-17 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/13104#discussion_r63584448 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala --- @@ -65,6 +65,14 @@ class TextSuite extends QueryTest

[GitHub] spark pull request: [SPARK-15323] Fix reading of partitioned forma...

2016-05-17 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/13104#discussion_r63584539 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/text/TextSuite.scala --- @@ -65,6 +65,14 @@ class TextSuite extends QueryTest

[GitHub] spark pull request: [WIP]Check SparkContext not stop

2016-05-17 Thread zsxwing
GitHub user zsxwing opened a pull request: https://github.com/apache/spark/pull/13154 [WIP]Check SparkContext not stop ## What changes were proposed in this pull request? (Please fill in changes proposed in this fix) ## How was this patch tested?

[GitHub] spark pull request: [SPARK-15317][Core]Don't store accumulators fo...

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

[GitHub] spark pull request: [WIP]Check SparkContext not stop

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

[GitHub] spark pull request: [WIP]Check SparkContext not stop

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

[GitHub] spark pull request: [WIP]Check SparkContext not stop

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

[GitHub] spark pull request: [WIP]Check SparkContext not stop

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13154#issuecomment-219827393 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: [WIP]Check SparkContext not stop

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

[GitHub] spark pull request: [SPARK-15339] [ML] ML 2.0 QA: Scala APIs and c...

2016-05-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/13129#discussion_r63588818 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/LinearRegression.scala --- @@ -252,7 +250,7 @@ class LinearRegression @Since("1.3.0")

[GitHub] spark pull request: [SPARK-15339] [ML] ML 2.0 QA: Scala APIs and c...

2016-05-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/13129#discussion_r63588786 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/AFTSurvivalRegression.scala --- @@ -88,8 +88,8 @@ private[regression] trait

[GitHub] spark pull request: [SPARK-15339] [ML] ML 2.0 QA: Scala APIs and c...

2016-05-17 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/13129#discussion_r63588807 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/IsotonicRegression.scala --- @@ -69,8 +69,8 @@ private[regression] trait

[GitHub] spark pull request: [SPARK-14615][ML] Use the new ML Vector and Ma...

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

[GitHub] spark pull request: [SPARK-14615][ML] Use the new ML Vector and Ma...

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

[GitHub] spark pull request: [SPARK-14615][ML] Use the new ML Vector and Ma...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12627#issuecomment-219832495 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-15307] [SQL] speed up listing files for...

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

[GitHub] spark pull request: [SPARK-15307] [SQL] speed up listing files for...

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

[GitHub] spark pull request: [SPARK-14615][ML] Use the new ML Vector and Ma...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12627 --- 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-15307] [SQL] speed up listing files for...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13094#issuecomment-219833828 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-14615][ML] Use the new ML Vector and Ma...

2016-05-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/12627#issuecomment-219834198 LGTM. Merged into master and branch-2.0. This should complete the major MLlib API changes in 2.0. Thanks! On retrospective, I think we under-estimated the

[GitHub] spark pull request: [SPARK-15244][PYTHON] Type of column name crea...

2016-05-17 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/13097#issuecomment-219835499 Hi, @davies . Could you merge 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

[GitHub] spark pull request: [SPARK-15317][Core]Don't store accumulators fo...

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

[GitHub] spark pull request: [SPARK-15317][Core]Don't store accumulators fo...

2016-05-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/13153#issuecomment-219836046 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-15317][Core]Don't store accumulators fo...

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

[GitHub] spark pull request: [SPARK-15244][PYTHON] Type of column name crea...

2016-05-17 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/13097#issuecomment-219836640 LGTM, Merging this into master and 2.0, 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

[GitHub] spark pull request: [SPARK-15244][PYTHON] Type of column name crea...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/13097 --- 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-15244][PYTHON] Type of column name crea...

2016-05-17 Thread dongjoon-hyun
Github user dongjoon-hyun commented on the pull request: https://github.com/apache/spark/pull/13097#issuecomment-219837638 Thank you, @davies ! --- 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-15317][Core]Don't store accumulators fo...

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

<    1   2   3   4   5   6   >