[GitHub] spark pull request: [SPARK-7889] [CORE] HistoryServer to refresh c...

2016-02-08 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/6935#discussion_r52184854 --- Diff: core/src/main/scala/org/apache/spark/deploy/history/ApplicationCache.scala --- @@ -0,0 +1,668 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-13139][SQL][WIP] Create native DDL comm...

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

[GitHub] spark pull request: [SPARK-13225] [SQL] Support Intersect All/Dist...

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

[GitHub] spark pull request: [SPARK-5095] [Mesos] Support launching multipl...

2016-02-08 Thread tnachen
Github user tnachen commented on a diff in the pull request: https://github.com/apache/spark/pull/10993#discussion_r52188015 --- Diff: core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/CoarseMesosSchedulerBackend.scala --- @@ -260,113 +257,208 @@ private[spark] class

[GitHub] spark pull request: [SPARK-13221] [SQL] Fixing GroupingSets when A...

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

[GitHub] spark pull request: [SPARK-13233][SQL][WIP] Python Dataset

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

[GitHub] spark pull request: [SPARK-13201][SPARK-13200] Deprecation warning...

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

[GitHub] spark pull request: [SPARK-13201][SPARK-13200] Deprecation warning...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/2#issuecomment-181248395 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-13201][SPARK-13200] Deprecation warning...

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

[GitHub] spark pull request: [SPARK-12792][SPARKR] Refactor RRDD to support...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10947#issuecomment-181250046 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-12792][SPARKR] Refactor RRDD to support...

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

[GitHub] spark pull request: [SPARK-12792][SPARKR] Refactor RRDD to support...

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

[GitHub] spark pull request: [SPARK-13221] [SQL] Fixing GroupingSets when A...

2016-02-08 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/11100#issuecomment-181436922 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-12725] [SQL] Resolving Name Conflicts i...

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

[GitHub] spark pull request: [SPARK-7889] [CORE] HistoryServer to refresh c...

2016-02-08 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/6935#issuecomment-181426395 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

[GitHub] spark pull request: [SPARK-13139][SQL][WIP] Create native DDL comm...

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

[GitHub] spark pull request: [SPARK-13225] [SQL] Support Intersect All/Dist...

2016-02-08 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/11106#issuecomment-181436898 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-7889] [CORE] HistoryServer to refresh c...

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

[GitHub] spark pull request: [SPARK-7889] [CORE] HistoryServer to refresh c...

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

[GitHub] spark pull request: [SPARK-13233][SQL][WIP] Python Dataset

2016-02-08 Thread cloud-fan
GitHub user cloud-fan opened a pull request: https://github.com/apache/spark/pull/7 [SPARK-13233][SQL][WIP] Python Dataset draft prototype, submit PR to test it via jenkins. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] spark pull request: [SPARK-5095] [Mesos] Support launching multipl...

2016-02-08 Thread tnachen
Github user tnachen commented on the pull request: https://github.com/apache/spark/pull/10993#issuecomment-181444389 Just one comment, overall 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

[GitHub] spark pull request: [SPARK-5095] [Mesos] Support launching multipl...

2016-02-08 Thread tnachen
Github user tnachen commented on a diff in the pull request: https://github.com/apache/spark/pull/10993#discussion_r52188465 --- Diff: docs/configuration.md --- @@ -825,13 +825,18 @@ Apart from these, the following properties are also available, and may be useful

[GitHub] spark pull request: [SPARK-7889] [CORE] HistoryServer to refresh c...

2016-02-08 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/6935#discussion_r52184490 --- Diff: core/src/main/scala/org/apache/spark/deploy/history/ApplicationCache.scala --- @@ -0,0 +1,668 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-12725] [SQL] Resolving Name Conflicts i...

2016-02-08 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/11050#issuecomment-181436878 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-13139][SQL][WIP] Create native DDL comm...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11048#issuecomment-181437759 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-12986][Doc] Fix pydoc warnings in mllib...

2016-02-08 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/11025#issuecomment-181265300 looks good to me as well and seems to pass pylint now @mengxr --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] spark pull request: [SPARK-13177][Examples]Update ActorWordCount e...

2016-02-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/3#discussion_r52155123 --- Diff: examples/src/main/scala/org/apache/spark/examples/streaming/ActorWordCount.scala --- @@ -63,7 +61,7 @@ class FeederActor extends Actor {

[GitHub] spark pull request: [SPARK-12469][CORE][WIP/RFC] Consistent accumu...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11105#issuecomment-181327618 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-12469][CORE][WIP/RFC] Consistent accumu...

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

[GitHub] spark pull request: [SPARK-13177][Examples]Update ActorWordCount e...

2016-02-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/3#discussion_r52163789 --- Diff: examples/src/main/scala/org/apache/spark/examples/streaming/ActorWordCount.scala --- @@ -39,7 +39,7 @@ case class

[GitHub] spark pull request: [SPARK-12982][SQL] Add table name validation i...

2016-02-08 Thread jayadevanmurali
Github user jayadevanmurali commented on the pull request: https://github.com/apache/spark/pull/11051#issuecomment-181266259 Thanks @hvanhovell for this info --- 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-13177][Examples]Update ActorWordCount e...

2016-02-08 Thread agsachin
GitHub user agsachin opened a pull request: https://github.com/apache/spark/pull/3 [SPARK-13177][Examples]Update ActorWordCount example to not directly use low level linked list as it is deprecated. You can merge this pull request into a Git repository by running: $ git

[GitHub] spark pull request: [SPARK-13065] [Streaming] streaming-twitter pa...

2016-02-08 Thread BenFradet
Github user BenFradet commented on the pull request: https://github.com/apache/spark/pull/11003#issuecomment-181302050 I invite you to read #1717 as this has been proposed and rejected before. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-13177][Examples]Update ActorWordCount e...

2016-02-08 Thread agsachin
Github user agsachin commented on a diff in the pull request: https://github.com/apache/spark/pull/3#discussion_r52155735 --- Diff: examples/src/main/scala/org/apache/spark/examples/streaming/ActorWordCount.scala --- @@ -18,16 +18,14 @@ // scalastyle:off println

[GitHub] spark pull request: [SPARK-12807] [YARN] Spark External Shuffle no...

2016-02-08 Thread steveloughran
Github user steveloughran commented on a diff in the pull request: https://github.com/apache/spark/pull/10780#discussion_r52160665 --- Diff: network/yarn/pom.xml --- @@ -86,6 +94,15 @@ + + +

[GitHub] spark pull request: [SPARK-13177][Examples]Update ActorWordCount e...

2016-02-08 Thread agsachin
Github user agsachin commented on the pull request: https://github.com/apache/spark/pull/3#issuecomment-181349834 @srowen if this is ok can u ask jenkins 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

[GitHub] spark pull request: [SPARK-12153][SPARK-7617][MLlib]add support of...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10152#issuecomment-181271212 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-12807] [YARN] Spark External Shuffle no...

2016-02-08 Thread steveloughran
Github user steveloughran commented on a diff in the pull request: https://github.com/apache/spark/pull/10780#discussion_r52161121 --- Diff: network/yarn/pom.xml --- @@ -96,6 +113,61 @@ + + + +

[GitHub] spark pull request: [SPARK-13065] [Streaming] streaming-twitter pa...

2016-02-08 Thread agsachin
Github user agsachin commented on the pull request: https://github.com/apache/spark/pull/11003#issuecomment-181354203 @BenFradet That's was a long thread , after reading all still I am not able to make conclusion what is final decision. There is no package in spark

[GitHub] spark pull request: [SPARK-12153][SPARK-7617][MLlib]add support of...

2016-02-08 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/10152#discussion_r52141782 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/feature/Word2Vec.scala --- @@ -289,26 +301,24 @@ class Word2Vec extends Serializable with Logging {

[GitHub] spark pull request: [SPARK-12153][SPARK-7617][MLlib]add support of...

2016-02-08 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/10152#discussion_r52141876 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/feature/Word2Vec.scala --- @@ -551,12 +553,17 @@ class Word2VecModel private[spark] ( }

[GitHub] spark pull request: [SPARK-12153][SPARK-7617][MLlib]add support of...

2016-02-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10152#discussion_r52142152 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/feature/Word2Vec.scala --- @@ -551,12 +553,17 @@ class Word2VecModel private[spark] ( }

[GitHub] spark pull request: [SPARK-12469][CORE][WIP/RFC] Consistent accumu...

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

[GitHub] spark pull request: [SPARK-13177][Examples]Update ActorWordCount e...

2016-02-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/3#discussion_r52154782 --- Diff: examples/src/main/scala/org/apache/spark/examples/streaming/ActorWordCount.scala --- @@ -39,7 +37,7 @@ case class

[GitHub] spark pull request: [SPARK-13177][Examples]Update ActorWordCount e...

2016-02-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/3#discussion_r52154706 --- Diff: examples/src/main/scala/org/apache/spark/examples/streaming/ActorWordCount.scala --- @@ -18,16 +18,14 @@ // scalastyle:off println

[GitHub] spark pull request: [SPARK-12469][CORE][WIP/RFC] Consistent accumu...

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

[GitHub] spark pull request: [SPARK-12807] [YARN] Spark External Shuffle no...

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

[GitHub] spark pull request: [SPARK-12153][SPARK-7617][MLlib]add support of...

2016-02-08 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10152#issuecomment-181253115 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-12153][SPARK-7617][MLlib]add support of...

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

[GitHub] spark pull request: [SPARK-12982][SQL] Add table name validation i...

2016-02-08 Thread jayadevanmurali
Github user jayadevanmurali commented on the pull request: https://github.com/apache/spark/pull/11051#issuecomment-181260259 @SparkQA Could you please do a test again @hvanhovell @davies Incorporated review comments in test case. Could you please review the same

[GitHub] spark pull request: [SPARK-12177] [STREAMING] Update KafkaDStreams...

2016-02-08 Thread markgrover
Github user markgrover commented on the pull request: https://github.com/apache/spark/pull/10953#issuecomment-181264407 Thanks for your feedback, @harishreedharan I have done a significant refactoring of code based on your feedback, sharing code where possible. The new RDD, is now

[GitHub] spark pull request: [SPARK-13177][Examples]Update ActorWordCount e...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/3#issuecomment-181299450 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-12177] [STREAMING] Update KafkaDStreams...

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

[GitHub] spark pull request: [SPARK-12177] [STREAMING] Update KafkaDStreams...

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

[GitHub] spark pull request: [SPARK-12153][SPARK-7617][MLlib]add support of...

2016-02-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10152#discussion_r52142120 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/feature/Word2Vec.scala --- @@ -289,26 +301,24 @@ class Word2Vec extends Serializable with Logging {

[GitHub] spark pull request: [SPARK-13012] [Documentation] Replace example ...

2016-02-08 Thread devaraj-kavali
Github user devaraj-kavali commented on the pull request: https://github.com/apache/spark/pull/11053#issuecomment-181258525 Thanks @yinxusen for review, I have fixed the comments, Please have a look into this. --- If your project is set up for it, you can reply to this email and

[GitHub] spark pull request: [SPARK-13165][STREAMING] Replace deprecated sy...

2016-02-08 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/11067#issuecomment-181267861 Yah @tedyu has a follow up PR that adds the scalastyle rule ready to rebase once this gets merged in. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-13177][Examples]Update ActorWordCount e...

2016-02-08 Thread agsachin
Github user agsachin commented on a diff in the pull request: https://github.com/apache/spark/pull/3#discussion_r52155740 --- Diff: examples/src/main/scala/org/apache/spark/examples/streaming/ActorWordCount.scala --- @@ -63,7 +61,7 @@ class FeederActor extends Actor {

[GitHub] spark pull request: [SPARK-7848] [Streaming] [Update SparkStreamin...

2016-02-08 Thread nirmannarang
GitHub user nirmannarang opened a pull request: https://github.com/apache/spark/pull/4 [SPARK-7848] [Streaming] [Update SparkStreaming docs to incorporate i…mll;'l. …mportant points] You can merge this pull request into a Git repository by running: $ git pull

[GitHub] spark pull request: [SPARK-12982][SQL] Add table name validation i...

2016-02-08 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/11051#discussion_r52145256 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala --- @@ -1291,4 +1291,19 @@ class DataFrameSuite extends QueryTest with

[GitHub] spark pull request: [SPARK-12982][SQL] Add table name validation i...

2016-02-08 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/11051#issuecomment-181265540 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] spark pull request: [SPARK-12982][SQL] Add table name validation i...

2016-02-08 Thread jayadevanmurali
Github user jayadevanmurali commented on the pull request: https://github.com/apache/spark/pull/11051#issuecomment-181269499 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-13177][Examples]Update ActorWordCount e...

2016-02-08 Thread agsachin
Github user agsachin commented on a diff in the pull request: https://github.com/apache/spark/pull/3#discussion_r52157133 --- Diff: examples/src/main/scala/org/apache/spark/examples/streaming/ActorWordCount.scala --- @@ -18,16 +18,14 @@ // scalastyle:off println

[GitHub] spark pull request: [SPARK-12177] [STREAMING] Update KafkaDStreams...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10953#issuecomment-181328631 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 project does

[GitHub] spark pull request: [SPARK-7848] [Streaming] [Update SparkStreamin...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4#issuecomment-18199 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-12982][SQL] Add table name validation i...

2016-02-08 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/11051#issuecomment-181265973 @jayadevanmurali you can request a retest by making a `retest this please` comment. The Jenkins PR builder will pick this up. See

[GitHub] spark pull request: [SPARK-12177] [STREAMING] Update KafkaDStreams...

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

[GitHub] spark pull request: [SPARK-12982][SQL] Add table name validation i...

2016-02-08 Thread jayadevanmurali
Github user jayadevanmurali commented on the pull request: https://github.com/apache/spark/pull/11051#issuecomment-181265819 @hvanhovell I will make that change now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-12153][SPARK-7617][MLlib]add support of...

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

[GitHub] spark pull request: [SPARK-12153][SPARK-7617][MLlib]add support of...

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

[GitHub] spark pull request: [SPARK-12982][SQL] Add table name validation i...

2016-02-08 Thread hvanhovell
Github user hvanhovell commented on the pull request: https://github.com/apache/spark/pull/11051#issuecomment-181296056 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

[GitHub] spark pull request: [SPARK-12503][SPARK-12505] Limit pushdown in U...

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

[GitHub] spark pull request: [SPARK-12177] [STREAMING] Update KafkaDStreams...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10953#issuecomment-181672442 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-12177] [STREAMING] Update KafkaDStreams...

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

[GitHub] spark pull request: [SPARK-13146][SQL] Management API for continuo...

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

[GitHub] spark pull request: [SPARK-13146][SQL] Management API for continuo...

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

[GitHub] spark pull request: [SPARK-13146][SQL] Management API for continuo...

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

[GitHub] spark pull request: [SPARK-13146][SQL] Management API for continuo...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11030#issuecomment-181681968 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 project does

[GitHub] spark pull request: [SPARK-13146][SQL] Management API for continuo...

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

[GitHub] spark pull request: [SPARK-13146][SQL] Management API for continuo...

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

[GitHub] spark pull request: [SPARK-13074][Core] Add JavaSparkContext. getP...

2016-02-08 Thread flyjy
Github user flyjy commented on the pull request: https://github.com/apache/spark/pull/10978#issuecomment-181688265 @srowen Thank you very much for your suggestions. The way you suggested is clearer but the previous PR is relatively simpler. The existing codes have passed the test

[GitHub] spark pull request: [SPARK-13146][SQL] Management API for continuo...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11030#issuecomment-181687891 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-13074][Core] Add JavaSparkContext. getP...

2016-02-08 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/10978#issuecomment-181739481 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

[GitHub] spark pull request: [SPARK-13074][Core] Add JavaSparkContext. getP...

2016-02-08 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/10978#discussion_r52273190 --- Diff: core/src/main/scala/org/apache/spark/api/java/JavaSparkContext.scala --- @@ -774,6 +774,16 @@ class JavaSparkContext(val sc: SparkContext)

[GitHub] spark pull request: [SPARK-13146][SQL] Management API for continuo...

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

[GitHub] spark pull request: [SPARK-12982][SQL] Add table name validation i...

2016-02-08 Thread jayadevanmurali
Github user jayadevanmurali commented on the pull request: https://github.com/apache/spark/pull/11051#issuecomment-181701118 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-12992][SQL] Support vectorized decoding...

2016-02-08 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/11055#issuecomment-181727263 LGTM, merging this 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

[GitHub] spark pull request: [SPARK-13074][Core] Add JavaSparkContext. getP...

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

[GitHub] spark pull request: [SPARK-13186][Streaming]Migrate away from Sync...

2016-02-08 Thread huaxingao
Github user huaxingao commented on the pull request: https://github.com/apache/spark/pull/11104#issuecomment-181664339 Sorry for the file line length problem. Fixed. --- 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-13154][PYTHON] Add linting for pydocs

2016-02-08 Thread holdenk
Github user holdenk commented on the pull request: https://github.com/apache/spark/pull/11109#issuecomment-181663977 cc @mengxr & @jkbradley --- 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: [WebUI][SPARK-7889] HistoryServer updates UI f...

2016-02-08 Thread squito
Github user squito commented on a diff in the pull request: https://github.com/apache/spark/pull/8#discussion_r52264218 --- Diff: core/src/main/scala/org/apache/spark/deploy/history/ApplicationCache.scala --- @@ -0,0 +1,669 @@ +/* + * Licensed to the Apache Software

[GitHub] spark pull request: [SPARK-13015][Docs] Replace example code in ml...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11128#issuecomment-181698382 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-13225] [SQL] Support Intersect All/Dist...

2016-02-08 Thread hvanhovell
Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/11106#discussion_r52272979 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala --- @@ -223,16 +222,22 @@ object HiveTypeCoercion {

[GitHub] spark pull request: [SPARK-13232][YARN] Fix executor node label

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11129#issuecomment-181737645 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-11701] dynamic allocation and speculati...

2016-02-08 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/10951#discussion_r52258455 --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala --- @@ -1153,7 +1153,16 @@ class DAGScheduler( } if

[GitHub] spark pull request: [SPARK-13186][Streaming]Migrate away from Sync...

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

[GitHub] spark pull request: [SPARK-13186][Streaming]Migrate away from Sync...

2016-02-08 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11104#issuecomment-181666297 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-13146][SQL] Management API for continuo...

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

[GitHub] spark pull request: [SPARK-13231] Make count failed values a user ...

2016-02-08 Thread ScrapCodes
Github user ScrapCodes commented on the pull request: https://github.com/apache/spark/pull/5#issuecomment-181716006 @JoshRosen Thanks for taking a look. I will update it, did you also consider renaming - or may be just updating the doc suffices ? --- If your project is set up

[GitHub] spark pull request: [SPARK-13232][YARN] Fix executor node label

2016-02-08 Thread AtkinsChang
GitHub user AtkinsChang opened a pull request: https://github.com/apache/spark/pull/11129 [SPARK-13232][YARN] Fix executor node label Specify node label for executor will cause Spark not working on Yarn due to Yarn does not support node container request with both label and

  1   2   3   4   5   >