[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread vanzin
Github user vanzin commented on a diff in the pull request: https://github.com/apache/spark/pull/4066#discussion_r24039211 --- Diff: core/src/main/scala/org/apache/spark/CommitDeniedException.scala --- @@ -0,0 +1,32 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-5155] [PySpark] [Streaming] Mqtt stream...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4229#issuecomment-72733949 [Test build #26668 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26668/consoleFull) for PR 4229 at commit

[GitHub] spark pull request: [SPARK-5182] [SPARK-5528] [SPARK-5509] [SPARK-...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4308#issuecomment-72734625 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: SPARK-5548: Fixed a race condition in AkkaUtil...

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

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4331#issuecomment-72619816 [Test build #26635 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26635/consoleFull) for PR 4331 at commit

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4331#issuecomment-72619824 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

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

[GitHub] spark pull request: [SPARK-5278][SQL] complete the check of ambigu...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4068#issuecomment-72622614 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5555] Enable UISeleniumSuite tests

2015-02-03 Thread JoshRosen
GitHub user JoshRosen opened a pull request: https://github.com/apache/spark/pull/4334 [SPARK-] Enable UISeleniumSuite tests This patch enables UISeleniumSuite, a set of tests for the Spark application web UI. These tests were previously disabled because they were slow, but I

[GitHub] spark pull request: [SPARK-5278][SQL] complete the check of ambigu...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4068#issuecomment-72622599 [Test build #26638 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26638/consoleFull) for PR 4068 at commit

[GitHub] spark pull request: [SPARK-5555] Enable UISeleniumSuite tests

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4334#issuecomment-72633082 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4331#issuecomment-72631120 [Test build #26649 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26649/consoleFull) for PR 4331 at commit

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4331#issuecomment-72631133 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-4795][Core] Redesign the primitive typ...

2015-02-03 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/3642#issuecomment-72631442 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-4795][Core] Redesign the primitive typ...

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

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4331#issuecomment-72632209 [Test build #575 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/575/consoleFull) for PR 4331 at commit

[GitHub] spark pull request: [SPARK-2945][YARN][Doc]add doc for spark.execu...

2015-02-03 Thread WangTaoTheTonic
GitHub user WangTaoTheTonic opened a pull request: https://github.com/apache/spark/pull/4335 [SPARK-2945][YARN][Doc]add doc for spark.executor.instances https://issues.apache.org/jira/browse/SPARK-2945 `spark.executor.instances` works. As this JIRA recommended, we should

[GitHub] spark pull request: [SPARK-5555] Enable UISeleniumSuite tests

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4334#issuecomment-72633068 [Test build #26650 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26650/consoleFull) for PR 4334 at commit

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72633362 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-2945][YARN][Doc]add doc for spark.execu...

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

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72633350 **[Test build #26645 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26645/consoleFull)** for PR 4066 at commit

[GitHub] spark pull request: [SPARK-4508] [SQL] build native date type to c...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4325#issuecomment-72625660 [Test build #26642 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26642/consoleFull) for PR 4325 at commit

[GitHub] spark pull request: [SPARK-4508] [SQL] build native date type to c...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4325#issuecomment-72625668 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SQL] DataFrame API update

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4332#issuecomment-72626851 [Test build #26644 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26644/consoleFull) for PR 4332 at commit

[GitHub] spark pull request: [SQL] DataFrame API update

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4332#issuecomment-72626858 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72627568 **[Test build #26637 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26637/consoleFull)** for PR 4066 at commit

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72627578 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: Minor: Fix TaskContext deprecated annotations.

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4333#issuecomment-72627902 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: Minor: Fix TaskContext deprecated annotations.

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4333#issuecomment-72627893 [Test build #26648 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26648/consoleFull) for PR 4333 at commit

[GitHub] spark pull request: [SQL][Hiveconsole] Bring hive console code up ...

2015-02-03 Thread OopsOutOfMemory
Github user OopsOutOfMemory commented on the pull request: https://github.com/apache/spark/pull/4330#issuecomment-72628043 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: [SQL] Minor changes for DataFrame Implementati...

2015-02-03 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/4339#issuecomment-72762303 Hey @chenghao-intel thanks for doing this. I'm making changes to this file right now so I will roll your change in. --- If your project is set up for it, you can reply to

[GitHub] spark pull request: [SPARK-5574] use given name prefix in dir

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4344#issuecomment-72762127 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5574] use given name prefix in dir

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4344#issuecomment-72762121 [Test build #26681 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26681/consoleFull) for PR 4344 at commit

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72762489 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72762480 [Test build #26682 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26682/consoleFull) for PR 4066 at commit

[GitHub] spark pull request: [SQL][DataFrame] defineUDF.

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

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4331#issuecomment-72763376 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4331#issuecomment-72763371 [Test build #26679 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26679/consoleFull) for PR 4331 at commit

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

2015-02-03 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/4331#issuecomment-72763611 Thanks. Merging 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 does not have

[GitHub] spark pull request: [WIP] [SPARK-4587] [mllib] ML model import/exp...

2015-02-03 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/4233#discussion_r24052353 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/classification/LogisticRegression.scala --- @@ -17,14 +17,17 @@ package

[GitHub] spark pull request: [WIP] [SPARK-4587] [mllib] ML model import/exp...

2015-02-03 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/4233#discussion_r24052532 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/classification/NaiveBayes.scala --- @@ -65,6 +67,70 @@ class NaiveBayesModel private[mllib] (

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72763987 Also, this could probably benefit from a more comprehensive unit test of the commit coordinator itself, since I think it's possible to alter the current implementation

[GitHub] spark pull request: [SPARK-5341] Use maven coordinates as dependen...

2015-02-03 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/4215#issuecomment-72763888 LGTM pending tests. --- 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-5460][MLlib] Wrapped `Try` around `dele...

2015-02-03 Thread x1-
GitHub user x1- opened a pull request: https://github.com/apache/spark/pull/4347 [SPARK-5460][MLlib] Wrapped `Try` around `deleteAllCheckpoints` - RandomForest. Because `deleteAllCheckpoints` has IOException potential. fix issue. You can merge this pull request into a Git

[GitHub] spark pull request: [SPARK-5460][MLlib] Wrapped `Try` around `dele...

2015-02-03 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/4347#discussion_r24053119 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/tree/RandomForest.scala --- @@ -244,7 +245,10 @@ private class RandomForest ( // Delete

[GitHub] spark pull request: [SQL] Minor changes for DataFrame Implementati...

2015-02-03 Thread chenghao-intel
Github user chenghao-intel closed the pull request at: https://github.com/apache/spark/pull/4339 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] spark pull request: [SQL] Minor changes for DataFrame Implementati...

2015-02-03 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/4339#issuecomment-72765655 OK, I am closing it. --- 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: [SQL] Minor changes for dataframe implementati...

2015-02-03 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4336#discussion_r24053791 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameImpl.scala --- @@ -260,11 +260,11 @@ private[sql] class DataFrameImpl protected[sql](

[GitHub] spark pull request: [SPARK-5526][SQL] fix issue about cast to date

2015-02-03 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/4307#issuecomment-72767656 BTW the pull request no longer merges cleanly with the master branch. Would be great to update. Thanks! --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-5526][SQL] fix issue about cast to date

2015-02-03 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/4307#issuecomment-72767614 Can you add a test case for this? --- 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: [WIP] [SPARK-4587] [mllib] ML model import/exp...

2015-02-03 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/4233#discussion_r24054409 --- Diff: mllib/src/test/scala/org/apache/spark/mllib/classification/LogisticRegressionSuite.scala --- @@ -425,25 +427,25 @@ class LogisticRegressionSuite

[GitHub] spark pull request: [WIP] [SPARK-4587] [mllib] ML model import/exp...

2015-02-03 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/4233#discussion_r24054690 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/classification/NaiveBayes.scala --- @@ -65,6 +67,70 @@ class NaiveBayesModel private[mllib] (

[GitHub] spark pull request: [WIP] [SPARK-4587] [mllib] ML model import/exp...

2015-02-03 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/4233#discussion_r24054729 --- Diff: mllib/src/test/scala/org/apache/spark/mllib/classification/LogisticRegressionSuite.scala --- @@ -425,25 +427,25 @@ class LogisticRegressionSuite

[GitHub] spark pull request: [SPARK-5520][MLlib] Make FP-Growth implementat...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4340#issuecomment-72769602 [Test build #26689 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26689/consoleFull) for PR 4340 at commit

[GitHub] spark pull request: [SPARK-5520][MLlib] Make FP-Growth implementat...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4340#issuecomment-72769609 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72770487 Still, though, I'd like to consider whether there's a good way to fix this in order to guarantee that requests to commit output for a stage occur after the stage

[GitHub] spark pull request: [FIX][MLLIB] fix seed handling in Python GMM

2015-02-03 Thread mengxr
GitHub user mengxr opened a pull request: https://github.com/apache/spark/pull/4349 [FIX][MLLIB] fix seed handling in Python GMM If `seed` is `None` on the python side, it will pass in as a `null`. So we should use `java.lang.Long` instead of `Long` to take it. You can merge this

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

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

[GitHub] spark pull request: [SPARK-4964] [Streaming] Exactly-once semantic...

2015-02-03 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/3798#issuecomment-72770451 Okay here are the two options. 1. createRDD returns RDD[(K,V)] or RDD[R], and DStream.foreachRDD uses rdd.asInstanceOf[HasOffsetRanges] 2. createRDD returns

[GitHub] spark pull request: [SQL] Minor changes for dataframe implementati...

2015-02-03 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4336#discussion_r24055322 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameImpl.scala --- @@ -260,11 +260,11 @@ private[sql] class DataFrameImpl protected[sql](

[GitHub] spark pull request: [SPARK-5520][MLlib] Make FP-Growth implementat...

2015-02-03 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/4340#issuecomment-72770558 LGTM. Merged into master and branch-1.3. 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-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72770871 [Test build #26691 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26691/consoleFull) for PR 4066 at commit

[GitHub] spark pull request: [WIP] [SPARK-4587] [mllib] ML model import/exp...

2015-02-03 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/4233#discussion_r24055623 --- Diff: mllib/src/test/scala/org/apache/spark/mllib/classification/LogisticRegressionSuite.scala --- @@ -459,7 +461,41 @@ class LogisticRegressionSuite

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

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

[GitHub] spark pull request: [SPARK-5341] Use maven coordinates as dependen...

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

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72763768 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5158] [core] [security] Spark standalon...

2015-02-03 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/4106#issuecomment-72763704 Hey @mccheah - if you are too busy I think it's fine to let it slip past 1.3, given that there are still several unknowns. --- If your project is set up for it, you

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72763765 **[Test build #26677 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26677/consoleFull)** for PR 4066 at commit

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

2015-02-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/4331 --- 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-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72763703 There's one subtle, extremely-unlikely race condition that can still occur here and which I would like to fix: currently, we seem to assume that messages sent from the

[GitHub] spark pull request: [SPARK-5460][MLlib] Wrapped `Try` around `dele...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4347#issuecomment-72764883 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-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72765259 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72765252 [Test build #26686 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26686/consoleFull) for PR 4066 at commit

[GitHub] spark pull request: [SPARK-5579][SQL][DataFrame] Support for proje...

2015-02-03 Thread rxin
GitHub user rxin opened a pull request: https://github.com/apache/spark/pull/4348 [SPARK-5579][SQL][DataFrame] Support for project/filter using SQL expressions ```scala df.selectExpr(abs(colA), colB) df.filter(age 21) ``` You can merge this pull request into a Git

[GitHub] spark pull request: [SPARK-2996] Implement userClassPathFirst for ...

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

[GitHub] spark pull request: [SPARK-5579][SQL][DataFrame] Support for proje...

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

[GitHub] spark pull request: [SPARK-5574] use given name prefix in dir

2015-02-03 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/4344#issuecomment-72767317 Do you know what the common root is? Is it just tmp? If yes, maybe we do want to prefix spark always. If it is some special Spark tmp folder, then I think it is fine to not

[GitHub] spark pull request: [SQL] Minor changes for dataframe implementati...

2015-02-03 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/4336#discussion_r24053977 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameImpl.scala --- @@ -260,11 +260,11 @@ private[sql] class DataFrameImpl protected[sql](

[GitHub] spark pull request: [SPARK-5574] use given name prefix in dir

2015-02-03 Thread vanzin
Github user vanzin commented on the pull request: https://github.com/apache/spark/pull/4344#issuecomment-72768195 52f575 added a user-specific root under the temp dir, so you'd always have these directories under one that's named `spark-[uuid]`. --- If your project is set up for it,

[GitHub] spark pull request: [WIP] [SPARK-4587] [mllib] ML model import/exp...

2015-02-03 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/4233#discussion_r24054410 --- Diff: mllib/src/test/scala/org/apache/spark/mllib/classification/LogisticRegressionSuite.scala --- @@ -459,7 +461,41 @@ class LogisticRegressionSuite

[GitHub] spark pull request: [WIP] [SPARK-4587] [mllib] ML model import/exp...

2015-02-03 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/4233#discussion_r24054680 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/classification/LogisticRegression.scala --- @@ -17,14 +17,17 @@ package

[GitHub] spark pull request: [WIP] [SPARK-4587] [mllib] ML model import/exp...

2015-02-03 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/4233#discussion_r24054795 --- Diff: mllib/src/test/scala/org/apache/spark/mllib/classification/LogisticRegressionSuite.scala --- @@ -459,7 +461,41 @@ class LogisticRegressionSuite

[GitHub] spark pull request: [SPARK-5574] use given name prefix in dir

2015-02-03 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/4344#issuecomment-72769307 Then this 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

[GitHub] spark pull request: [SPARK-5578][SQL][DataFrame] Provide a conveni...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4345#issuecomment-72769530 [Test build #26687 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26687/consoleFull) for PR 4345 at commit

[GitHub] spark pull request: [SPARK-5578][SQL][DataFrame] Provide a conveni...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4345#issuecomment-72769536 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5578][SQL][DataFrame] Provide a conveni...

2015-02-03 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/4345#discussion_r24055018 --- Diff: mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala --- @@ -132,14 +132,14 @@ class LogisticRegressionModel

[GitHub] spark pull request: [SQL] Correct the default size of TimestampTyp...

2015-02-03 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4314#discussion_r24055002 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/types/dataTypes.scala --- @@ -402,7 +402,7 @@ case object DateType extends NativeType {

[GitHub] spark pull request: [SPARK-5578][SQL][DataFrame] Provide a conveni...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4345#issuecomment-72770311 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5578][SQL][DataFrame] Provide a conveni...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4345#issuecomment-72770304 [Test build #26688 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26688/consoleFull) for PR 4345 at commit

[GitHub] spark pull request: [SPARK-5520][MLlib] Make FP-Growth implementat...

2015-02-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/4340 --- 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-5578][SQL][DataFrame] Provide a conveni...

2015-02-03 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/4345#discussion_r24055407 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala --- @@ -26,9 +27,8 @@ import

[GitHub] spark pull request: [SQL] Minor changes for dataframe implementati...

2015-02-03 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/4336#discussion_r24055445 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameImpl.scala --- @@ -260,11 +260,11 @@ private[sql] class DataFrameImpl protected[sql](

[GitHub] spark pull request: [SPARK-4879] [WIP] Use driver to coordinate Ha...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4066#issuecomment-72770879 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [FIX][MLLIB] fix seed handling in Python GMM

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

[GitHub] spark pull request: [SQL] Minor changes for dataframe implementati...

2015-02-03 Thread marmbrus
Github user marmbrus commented on a diff in the pull request: https://github.com/apache/spark/pull/4336#discussion_r24055512 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameImpl.scala --- @@ -260,11 +260,11 @@ private[sql] class DataFrameImpl protected[sql](

[GitHub] spark pull request: [SQL] Minor changes for dataframe implementati...

2015-02-03 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/4336#discussion_r24055684 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/DataFrameImpl.scala --- @@ -260,11 +260,11 @@ private[sql] class DataFrameImpl protected[sql](

[GitHub] spark pull request: [SPARK-4964] [Streaming] Exactly-once semantic...

2015-02-03 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/3798#issuecomment-72617088 I took a pass through the public API. I'm not very familiar with Kafka so it was somewhat slow going. However, some reactions: 1. We should try to tighten,

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

2015-02-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4331#issuecomment-72617481 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

2015-02-03 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4331#issuecomment-72617479 [Test build #26643 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/26643/consoleFull) for PR 4331 at commit

[GitHub] spark pull request: [SPARK-5554] [SQL] [PySpark] add more tests fo...

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

<    1   2   3   4   5   6   7   >