[GitHub] spark pull request: [SPARK-4631] unit test for MQTT

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3844#issuecomment-68419761 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-3325][Streaming] Add a parameter to the...

2014-12-30 Thread watermen
Github user watermen commented on the pull request: https://github.com/apache/spark/pull/3237#issuecomment-68419717 @tdas I had fixed the conflicts, but I don't know why bracket on line 56 does not align with bracket on line 60 in MimaExcludes.scala. --- If your project is set up for

[GitHub] spark pull request: [SPARK-5007] [CORE] Try random port when start...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3845#issuecomment-68419679 [Test build #24930 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24930/consoleFull) for PR 3845 at commit [`114ec73`](https://githu

[GitHub] spark pull request: [SPARK-5006][Deploy]spark.port.maxRetries does...

2014-12-30 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on a diff in the pull request: https://github.com/apache/spark/pull/3841#discussion_r22373227 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -1691,15 +1691,12 @@ private[spark] object Utils extends Logging { /**

[GitHub] spark pull request: [SPARK-4988][SQL] Fix: 'Create table ..as sele...

2014-12-30 Thread guowei2
Github user guowei2 commented on the pull request: https://github.com/apache/spark/pull/3821#issuecomment-68419415 @marmbrus i figured out it in the issue. in this case, execution `TakeOrdered` invokes `convertRowToScala` to change `Decimal` into `BigDecimal`. i've tried to rem

[GitHub] spark pull request: [SPARK-5006][Deploy]spark.port.maxRetries does...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3841#issuecomment-68419355 [Test build #24928 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24928/consoleFull) for PR 3841 at commit [`396c226`](https://githu

[GitHub] spark pull request: [SPARK-3325][Streaming] Add a parameter to the...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3237#issuecomment-68419360 [Test build #24929 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24929/consoleFull) for PR 3237 at commit [`f6ac3cb`](https://githu

[GitHub] spark pull request: [SPARK-3325][Streaming] Add a parameter to the...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3237#issuecomment-68419177 [Test build #24927 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24927/consoleFull) for PR 3237 at commit [`3b9d5cf`](https://githu

[GitHub] spark pull request: [SPARK-5006][Deploy]spark.port.maxRetries does...

2014-12-30 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/3841#discussion_r22373048 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -1691,15 +1691,12 @@ private[spark] object Utils extends Logging { /**

[GitHub] spark pull request: [SPARK-5006][Deploy]spark.port.maxRetries does...

2014-12-30 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on a diff in the pull request: https://github.com/apache/spark/pull/3841#discussion_r22373041 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -1719,6 +1716,7 @@ private[spark] object Utils extends Logging { serv

[GitHub] spark pull request: [SPARK-794][Core] Remove sleep() in ClusterSch...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3851#issuecomment-68418799 [Test build #24926 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24926/consoleFull) for PR 3851 at commit [`04c3e64`](https://githu

[GitHub] spark pull request: [SPARK-5028][Streaming]Add total received and ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3852#issuecomment-68418798 [Test build #24925 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24925/consoleFull) for PR 3852 at commit [`c8c4877`](https://githu

[GitHub] spark pull request: [SPARK-5006][Deploy]spark.port.maxRetries does...

2014-12-30 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on a diff in the pull request: https://github.com/apache/spark/pull/3841#discussion_r22373021 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -1691,15 +1691,12 @@ private[spark] object Utils extends Logging { /**

[GitHub] spark pull request: [SPARK-5028][Streaming]Add total received and ...

2014-12-30 Thread jerryshao
GitHub user jerryshao opened a pull request: https://github.com/apache/spark/pull/3852 [SPARK-5028][Streaming]Add total received and processed records metrics to Streaming UI This is a follow-up work of [SPARK-4537](https://issues.apache.org/jira/browse/SPARK-4537). Adding total r

[GitHub] spark pull request: [SPARK-5006][Deploy]spark.port.maxRetries does...

2014-12-30 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on a diff in the pull request: https://github.com/apache/spark/pull/3841#discussion_r22372988 --- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala --- @@ -176,6 +176,10 @@ class SparkContext(config: SparkConf) extends Logging with

[GitHub] spark pull request: [SPARK-5000][SQL] Alias support string literal...

2014-12-30 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/3837#issuecomment-68418668 Hi @marmbrus, yes, standard sql support this, see this sql(one of TPCDS ) ![image](https://cloud.githubusercontent.com/assets/7018048/5584899/0363a754-90d6-11e4-93

[GitHub] spark pull request: [SPARK-794][Core] Remove sleep() in ClusterSch...

2014-12-30 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/3851#issuecomment-68418669 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 ha

[GitHub] spark pull request: [SPARK-4298][Core] - The spark-submit cannot r...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3561#issuecomment-68418650 [Test build #24924 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24924/consoleFull) for PR 3561 at commit [`14daa20`](https://githu

[GitHub] spark pull request: [SPARK-4298][Core] - The spark-submit cannot r...

2014-12-30 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/3561#issuecomment-68418611 I've merged #3739 into `master`, so this should be ready to retest. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] spark pull request: [SPARK-4298][Core] - The spark-submit cannot r...

2014-12-30 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/3561#issuecomment-68418615 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 ha

[GitHub] spark pull request: [SPARK-4465] runAsSparkUser doesn't affect Tas...

2014-12-30 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/spark/pull/3741#issuecomment-68418600 @tnachen Please review my patch, again. --- 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 d

[GitHub] spark pull request: [SPARK-5006][Deploy]spark.port.maxRetries does...

2014-12-30 Thread WangTaoTheTonic
Github user WangTaoTheTonic commented on the pull request: https://github.com/apache/spark/pull/3841#issuecomment-68418574 I set `spark.port.maxRetries` to 1 in spark-defaults.conf and ran 3 SparkPi example using yarn-client mode in one node. Logs in 3 drivers' side were: >14/12/31

[GitHub] spark pull request: [SPARK-1010] Clean up uses of System.setProper...

2014-12-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3739 --- 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 enab

[GitHub] spark pull request: [SPARK-4014] Change TaskContext.attemptId to r...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3849#issuecomment-68418485 [Test build #24923 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24923/consoleFull) for PR 3849 at commit [`cbe4d76`](https://githu

[GitHub] spark pull request: [Spark-2201] Improve FlumeInputDStream's stabi...

2014-12-30 Thread joyyoj
Github user joyyoj closed the pull request at: https://github.com/apache/spark/pull/1755 --- 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 enab

[GitHub] spark pull request: [Spark-2201] Improve FlumeInputDStream's stabi...

2014-12-30 Thread joyyoj
Github user joyyoj commented on the pull request: https://github.com/apache/spark/pull/1755#issuecomment-68418459 Never mind, very very sorry for replay so late. --- 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 pro

[GitHub] spark pull request: [SPARK-1010] Clean up uses of System.setProper...

2014-12-30 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/3739#issuecomment-68418439 I'm going to merge this into `master` (1.3.0) to unblock some PRs that are depending on this cleanup. I'll work on backports to the other branches, too, since some of

[GitHub] spark pull request: [SPARK-4014] Change TaskContext.attemptId to r...

2014-12-30 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/3849#discussion_r22372823 --- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala --- @@ -180,7 +189,7 @@ private[spark] class Executor( // Run the a

[GitHub] spark pull request: [SPARK-4764] Ensure that files are fetched ato...

2014-12-30 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/2855#discussion_r22372752 --- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala --- @@ -425,8 +425,7 @@ private[spark] object Utils extends Logging { conf: SparkCon

[GitHub] spark pull request: [SPARK-4631] unit test for MQTT

2014-12-30 Thread tdas
Github user tdas commented on a diff in the pull request: https://github.com/apache/spark/pull/3844#discussion_r22372739 --- Diff: external/mqtt/src/test/scala/org/apache/spark/streaming/mqtt/MQTTStreamSuite.scala --- @@ -17,31 +17,58 @@ package org.apache.spark.streami

[GitHub] spark pull request: [SPARK-4631] unit test for MQTT

2014-12-30 Thread tdas
Github user tdas commented on a diff in the pull request: https://github.com/apache/spark/pull/3844#discussion_r22372726 --- Diff: external/mqtt/src/test/scala/org/apache/spark/streaming/mqtt/MQTTStreamSuite.scala --- @@ -17,31 +17,58 @@ package org.apache.spark.streami

[GitHub] spark pull request: [SPARK-794][Core] Remove sleep() in ClusterSch...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3851#issuecomment-68417620 [Test build #24921 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24921/consoleFull) for PR 3851 at commit [`04c3e64`](https://gith

[GitHub] spark pull request: [SPARK-794][Core] Remove sleep() in ClusterSch...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3851#issuecomment-68417622 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4631] unit test for MQTT

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3844#issuecomment-68417591 [Test build #24922 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24922/consoleFull) for PR 3844 at commit [`e8b6623`](https://githu

[GitHub] spark pull request: [SPARK-4789] [SPARK-4942] [mllib] Standardize ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3637#issuecomment-68417484 [Test build #24920 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24920/consoleFull) for PR 3637 at commit [`255c7f3`](https://gith

[GitHub] spark pull request: [SPARK-4789] [SPARK-4942] [mllib] Standardize ...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3637#issuecomment-68417486 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4501][Core] - Create build/mvn to autom...

2014-12-30 Thread witgo
Github user witgo commented on the pull request: https://github.com/apache/spark/pull/3707#issuecomment-68417443 Seems to be the issue caused by the command line is killed. But,I tested both in the mac os x and centos, and could not reproduce the issue. --- If your project is set

[GitHub] spark pull request: [SPARK-4631] unit test for MQTT

2014-12-30 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/3844#issuecomment-68417426 Jenksin, this is ok to test. --- 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 th

[GitHub] spark pull request: Shading the Jetty dependency.

2014-12-30 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/2984#issuecomment-68417417 If this PR is not useful any more, mind closing 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 y

[GitHub] spark pull request: Spark 3754 spark streaming file system api cal...

2014-12-30 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/2703#issuecomment-68417363 Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled an

[GitHub] spark pull request: SPARK-2757 [BUILD] [STREAMING] Add Mima test f...

2014-12-30 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/3842#issuecomment-68417257 A quick pass shows me that `spark-streaming-flume` uses the following from the `spark-streaming-flume-sink` module - The generated Avro protocol classes - SparkFlumeProtc

[GitHub] spark pull request: SPARK-4921. TaskSetManager.dequeueTask returns...

2014-12-30 Thread CodingCat
Github user CodingCat commented on the pull request: https://github.com/apache/spark/pull/3816#issuecomment-68417133 Hi, @sryza, yes, NO_PREF tasks will be scheduled only when the maxLocality is NO_PREF. Now, I think it's safe to apply this change, since we have that

[GitHub] spark pull request: SPARK-2757 [BUILD] [STREAMING] Add Mima test f...

2014-12-30 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/3842#issuecomment-68417061 Just wondering what is necessity for binary compatibility here? I am guessing that one should be able to operation Spark Streaming 1.2 programs with a deployed Sink from 1.0

[GitHub] spark pull request: [SPARK-5003][SQL]cast support date data type

2014-12-30 Thread haiyangsea
Github user haiyangsea commented on a diff in the pull request: https://github.com/apache/spark/pull/3839#discussion_r22372432 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala --- @@ -29,13 +29,14 @@ import org.apache.spark.sql.TestData._ import org.a

[GitHub] spark pull request: [SPARK-4574][SQL] Adding support for defining ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3431#issuecomment-68416333 [Test build #24919 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24919/consoleFull) for PR 3431 at commit [`cf982d2`](https://gith

[GitHub] spark pull request: [SPARK-4574][SQL] Adding support for defining ...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3431#issuecomment-68416337 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: Spark 3754 spark streaming file system api cal...

2014-12-30 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/2703#issuecomment-68415747 I'll take a look this weekend when I have some time. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] spark pull request: [branch-1.0][SPARK-4355] ColumnStatisticsAggre...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3850#issuecomment-68415602 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [branch-1.0][SPARK-4355] ColumnStatisticsAggre...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3850#issuecomment-68415599 **[Test build #24914 timed out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24914/consoleFull)** for PR 3850 at commit [`ae9b94a`](https://git

[GitHub] spark pull request: SPARK-4156 [MLLIB] EM algorithm for GMMs

2014-12-30 Thread tgaloppo
Github user tgaloppo commented on the pull request: https://github.com/apache/spark/pull/3022#issuecomment-68415536 @jkbradley No problem. Let's start with 5020, and I'll move on from there. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-794][Core] Remove sleep() in ClusterSch...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3851#issuecomment-68415340 [Test build #24921 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24921/consoleFull) for PR 3851 at commit [`04c3e64`](https://githu

[GitHub] spark pull request: Spark 3754 spark streaming file system api cal...

2014-12-30 Thread tdas
Github user tdas commented on the pull request: https://github.com/apache/spark/pull/2703#issuecomment-68415266 Ping? --- 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

[GitHub] spark pull request: [SPARK-794][Core] Remove sleep() in ClusterSch...

2014-12-30 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/3851#issuecomment-68415123 Jenkins, test 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-4943][SQL] Allow table name having dot ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3848#issuecomment-68415108 [Test build #24918 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24918/consoleFull) for PR 3848 at commit [`00adf40`](https://gith

[GitHub] spark pull request: [SPARK-4943][SQL] Allow table name having dot ...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3848#issuecomment-68415109 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4789] [SPARK-4942] [mllib] Standardize ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3637#issuecomment-68415080 [Test build #24920 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24920/consoleFull) for PR 3637 at commit [`255c7f3`](https://githu

[GitHub] spark pull request: [SPARK-4937][SQL] Adding optimization to simpl...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3778#issuecomment-68414980 [Test build #24917 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24917/consoleFull) for PR 3778 at commit [`9570211`](https://gith

[GitHub] spark pull request: [SPARK-4937][SQL] Adding optimization to simpl...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3778#issuecomment-68414986 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-3218, SPARK-3219, SPARK-3261, SPARK-342...

2014-12-30 Thread derrickburns
Github user derrickburns commented on the pull request: https://github.com/apache/spark/pull/2634#issuecomment-68414436 No problem. Thx! Sent from my iPhone > On Dec 30, 2014, at 3:23 PM, Xiangrui Meng wrote: > > @derrickburns I'm going to check the implemen

[GitHub] spark pull request: SPARK-4156 [MLLIB] EM algorithm for GMMs

2014-12-30 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/3022#issuecomment-68414406 @tgaloppo It's ideal if we assign & fix one JIRA at a time (as separate PRs). Can I start by assigning one of your choosing? For 5018, there is only [one othe

[GitHub] spark pull request: [SAPRK-4967] [SQL] File name with comma will c...

2014-12-30 Thread chenghao-intel
Github user chenghao-intel closed the pull request at: https://github.com/apache/spark/pull/3800 --- 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

[GitHub] spark pull request: [SAPRK-4967] [SQL] File name with comma will c...

2014-12-30 Thread chenghao-intel
Github user chenghao-intel commented on the pull request: https://github.com/apache/spark/pull/3800#issuecomment-68413730 Yea, I agree and closing this PR. :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your projec

[GitHub] spark pull request: [SPARK-794][Core] Remove sleep() in ClusterSch...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3851#issuecomment-68412691 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 pro

[GitHub] spark pull request: [SPARK-794][Core] Remove sleep() in ClusterSch...

2014-12-30 Thread brennonyork
GitHub user brennonyork opened a pull request: https://github.com/apache/spark/pull/3851 [SPARK-794][Core] Remove sleep() in ClusterScheduler.stop Removed `sleep()` from the `stop()` method of the `TaskSchedulerImpl` class which, from the JIRA ticket, is believed to be a legacy arti

[GitHub] spark pull request: [SPARK-4574][SQL] Adding support for defining ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3431#issuecomment-68412433 [Test build #24919 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24919/consoleFull) for PR 3431 at commit [`cf982d2`](https://githu

[GitHub] spark pull request: [SPARK-4014] Change TaskContext.attemptId to r...

2014-12-30 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/3849#issuecomment-68412414 Hmm, looks like this failed MiMa checks due to the addition of a new method to a public interface: ``` [info] spark-core: found 1 potential binary incompati

[GitHub] spark pull request: [WIP][SPARK-3541][MLLIB] New ALS implementatio...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3720#issuecomment-68412287 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [WIP][SPARK-3541][MLLIB] New ALS implementatio...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3720#issuecomment-68412281 [Test build #24911 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24911/consoleFull) for PR 3720 at commit [`213d163`](https://gith

[GitHub] spark pull request: [SPARK-4998][MLlib]delete the "train" function

2014-12-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/3836 --- 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 enab

[GitHub] spark pull request: [SPARK-4998][MLlib]delete the "train" function

2014-12-30 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/3836#issuecomment-68412122 LGTM. Merged into master. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-4789] [SPARK-4942] [mllib] Standardize ...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3637#issuecomment-68411669 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4789] [SPARK-4942] [mllib] Standardize ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3637#issuecomment-68411664 [Test build #24913 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24913/consoleFull) for PR 3637 at commit [`1bb315f`](https://gith

[GitHub] spark pull request: [SPARK-4998][MLlib]delete the "train" function

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3836#issuecomment-68411538 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4998][MLlib]delete the "train" function

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3836#issuecomment-68411535 [Test build #24910 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24910/consoleFull) for PR 3836 at commit [`4e13133`](https://gith

[GitHub] spark pull request: [SPARK-4943][SQL] Allow table name having dot ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3848#issuecomment-68411286 [Test build #24918 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24918/consoleFull) for PR 3848 at commit [`00adf40`](https://githu

[GitHub] spark pull request: [SPARK-4789] [SPARK-4942] [mllib] Standardize ...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3637#issuecomment-68411084 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4789] [SPARK-4942] [mllib] Standardize ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3637#issuecomment-68411078 [Test build #24912 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24912/consoleFull) for PR 3637 at commit [`bb02582`](https://gith

[GitHub] spark pull request: [SPARK-4937][SQL] Adding optimization to simpl...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3778#issuecomment-68410976 [Test build #24917 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24917/consoleFull) for PR 3778 at commit [`9570211`](https://githu

[GitHub] spark pull request: [SPARK-4937][SQL] Adding optimization to simpl...

2014-12-30 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/3778#issuecomment-68410961 Sure, 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 feature ena

[GitHub] spark pull request: [SPARK-4014] Change TaskContext.attemptId to r...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3849#issuecomment-68410774 [Test build #24908 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24908/consoleFull) for PR 3849 at commit [`9d8d4d1`](https://gith

[GitHub] spark pull request: [SPARK-4014] Change TaskContext.attemptId to r...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3849#issuecomment-68410782 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4943][SQL] Allow table name having dot ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3848#issuecomment-68410695 [Test build #24916 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24916/consoleFull) for PR 3848 at commit [`623cf91`](https://gith

[GitHub] spark pull request: [SPARK-4943][SQL] Allow table name having dot ...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3848#issuecomment-68410696 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4574][SQL] Adding support for defining ...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3431#issuecomment-68410619 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-4943][SQL] Allow table name having dot ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3848#issuecomment-68410615 [Test build #24916 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24916/consoleFull) for PR 3848 at commit [`623cf91`](https://githu

[GitHub] spark pull request: [SPARK-4574][SQL] Adding support for defining ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3431#issuecomment-68410616 [Test build #24915 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24915/consoleFull) for PR 3431 at commit [`445b57b`](https://gith

[GitHub] spark pull request: [SPARK-4943][SQL] Allow table name having dot ...

2014-12-30 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/3848#issuecomment-68410581 ok to test --- 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 ena

[GitHub] spark pull request: [SPARK-4937][SQL] Adding optimization to simpl...

2014-12-30 Thread marmbrus
Github user marmbrus commented on the pull request: https://github.com/apache/spark/pull/3778#issuecomment-68410453 Oh I see, sorry that was not clear from the discussion. I think it would be best to do this as an augmentation to what is already merged in. --- If your project is set

[GitHub] spark pull request: [SPARK-4261][BUILD][SQL] Make right version in...

2014-12-30 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/3124#issuecomment-68410362 Ping --- 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 w

[GitHub] spark pull request: [SPARK-4237][BUILD] Fix MANIFEST.MF in maven a...

2014-12-30 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/3103#issuecomment-68410347 Ping --- 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 w

[GitHub] spark pull request: [SPARK-4998][MLlib]delete the "train" function

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3836#issuecomment-68410039 [Test build #24909 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24909/consoleFull) for PR 3836 at commit [`4e13133`](https://gith

[GitHub] spark pull request: [SPARK-4998][MLlib]delete the "train" function

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3836#issuecomment-68410041 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: [SPARK-3218, SPARK-3219, SPARK-3261, SPARK-342...

2014-12-30 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/2634#issuecomment-68409940 @derrickburns I'm going to check the implementation during the break. Since it is hard to locate the serialization issue, do you mind me making changes to your implementat

[GitHub] spark pull request: [SPARK-4574][SQL] Adding support for defining ...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3431#issuecomment-68409631 [Test build #24915 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24915/consoleFull) for PR 3431 at commit [`445b57b`](https://githu

[GitHub] spark pull request: [SPARK-4943][SQL] Allow table name having dot ...

2014-12-30 Thread alexliu68
Github user alexliu68 commented on the pull request: https://github.com/apache/spark/pull/3848#issuecomment-68409413 I rebase the commit to simply fixing the parsing issue and concatenate cluster name, database name and table name into a single string with dot in between. Now it can p

[GitHub] spark pull request: SPARK-4660: Use correct default classloader in...

2014-12-30 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3840#issuecomment-68408981 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24

[GitHub] spark pull request: SPARK-4660: Use correct default classloader in...

2014-12-30 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/3840#issuecomment-68408974 [QA tests have finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/24907/consoleFull) for PR 3840 at commit [`86bc5eb`](https://github.com/a

[GitHub] spark pull request: [SPARK-4574][SQL] Adding support for defining ...

2014-12-30 Thread scwf
Github user scwf commented on a diff in the pull request: https://github.com/apache/spark/pull/3431#discussion_r22369162 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/sources/PrunedScanSuite.scala --- @@ -19,10 +19,11 @@ package org.apache.spark.sql.sources impo

[GitHub] spark pull request: [SPARK-4574][SQL] Adding support for defining ...

2014-12-30 Thread scwf
Github user scwf commented on a diff in the pull request: https://github.com/apache/spark/pull/3431#discussion_r22369155 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/sources/ddl.scala --- @@ -99,8 +209,15 @@ private[sql] case class CreateTableUsing( sys.er

[GitHub] spark pull request: [SPARK-4871][SQL] Show sql statement in spark ...

2014-12-30 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/3718#issuecomment-68408513 @marmbrus, can this go since i have filed a PR to add pop-up for full job desc? --- If your project is set up for it, you can reply to this email and have your reply appea

<    1   2   3   4   5   >