[GitHub] spark pull request: [SPARK-8970][SQL] remove unnecessary abstracti...

2015-07-09 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/7339#issuecomment-120256576 cc @rxin --- 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-8706] [PySpark] [Project infra] Add pyl...

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

[GitHub] spark pull request: [SPARK-8969][SQL] move type-check from BinaryA...

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

[GitHub] spark pull request: [SPARK-8969][SQL] move type-check from BinaryA...

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

[GitHub] spark pull request: [SPARK-8706] [PySpark] [Project infra] Add pyl...

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

[GitHub] spark pull request: [SPARK-8969][SQL] move type-check from BinaryA...

2015-07-09 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/7338#issuecomment-120256223 retest this please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

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

2015-07-09 Thread prabeesh
Github user prabeesh commented on the pull request: https://github.com/apache/spark/pull/4229#issuecomment-120254773 @tdas I am working on it. Will update you soon --- 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 p

[GitHub] spark pull request: [SPARK-8221][SQL]Add pmod function

2015-07-09 Thread zhichao-li
Github user zhichao-li commented on a diff in the pull request: https://github.com/apache/spark/pull/6783#discussion_r34331212 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala --- @@ -387,3 +387,67 @@ case class MinOf(left: Expressio

[GitHub] spark pull request: [SPARK-6941][SQL] Provide a better error messa...

2015-07-09 Thread yijieshen
Github user yijieshen commented on the pull request: https://github.com/apache/spark/pull/7342#issuecomment-120251015 @yhuai it seems not possible to do checking in `CheckAnalysis` because LogicalRelation or LogicalRDD is not accessible there. Any ideas? --- If your project is set up

[GitHub] spark pull request: [SPARK-8271][SQL]string function: soundex

2015-07-09 Thread HuJiayin
Github user HuJiayin commented on the pull request: https://github.com/apache/spark/pull/7115#issuecomment-120250661 This patch fails MiMa tests. [error] running /home/jenkins/workspace/SparkPullRequestBuilder@2/dev/mima ; received return code 255 Can you help on this? Thanks m

[GitHub] spark pull request: [SPARK-6941][SQL] Provide a better error messa...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7342#issuecomment-120249687 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 pr

[GitHub] spark pull request: [SPARK-8271][SQL]string function: soundex

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7115#issuecomment-120249301 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 project

[GitHub] spark pull request: [SPARK-8271][SQL]string function: soundex

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7115#issuecomment-120249290 [Test build #37017 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37017/console) for PR 7115 at commit [`59322bc`](https://github.

[GitHub] spark pull request: [SPARK-6941][SQL] Provide a better error messa...

2015-07-09 Thread yijieshen
GitHub user yijieshen opened a pull request: https://github.com/apache/spark/pull/7342 [SPARK-6941][SQL] Provide a better error message to when inserting into RDD based table JIRA: https://issues.apache.org/jira/browse/SPARK-6941 You can merge this pull request into a Git repositor

[GitHub] spark pull request: [SPARK-8970][SQL] remove unnecessary abstracti...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7339#issuecomment-120248189 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 project

[GitHub] spark pull request: [SPARK-8923] [Documentation, MLlib] Add @since...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7341#issuecomment-120248151 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 pr

[GitHub] spark pull request: [SPARK-8840][SparkR] Add float coercion on Spa...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7280#issuecomment-120248126 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 project

[GitHub] spark pull request: [SPARK-8840][SparkR] Add float coercion on Spa...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7280#issuecomment-120248085 [Test build #37008 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37008/console) for PR 7280 at commit [`733015a`](https://github.

[GitHub] spark pull request: [SPARK-8970][SQL] remove unnecessary abstracti...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7339#issuecomment-120248120 [Test build #37010 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37010/console) for PR 7339 at commit [`394d3ac`](https://github.

[GitHub] spark pull request: [SPARK-8923] [Documentation, MLlib] Add @since...

2015-07-09 Thread rahulpalamuttam
GitHub user rahulpalamuttam opened a pull request: https://github.com/apache/spark/pull/7341 [SPARK-8923] [Documentation, MLlib] Add @since tags to mllib.fpm You can merge this pull request into a Git repository by running: $ git pull https://github.com/rahulpalamuttam/spark T

[GitHub] spark pull request: [SPARK-8881] Fix algorithm for scheduling exec...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7274#issuecomment-120247772 [Test build #37007 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37007/console) for PR 7274 at commit [`40c8f9f`](https://github.

[GitHub] spark pull request: [SPARK-8881] Fix algorithm for scheduling exec...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7274#issuecomment-120247808 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 project

[GitHub] spark pull request: [SPARK-8271][SQL]string function: soundex

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7115#issuecomment-120247329 [Test build #37017 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37017/consoleFull) for PR 7115 at commit [`59322bc`](https://gith

[GitHub] spark pull request: [SPARK-8443][SQL] Split GenerateMutableProject...

2015-07-09 Thread maropu
Github user maropu commented on the pull request: https://github.com/apache/spark/pull/7076#issuecomment-120247243 This fix is kind of hack things to me. It'd be better to check the code size and, if it is over 64KB (the janino limitation), throw an exception to fall back into

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] Binary processing so...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6444#issuecomment-120247317 [Test build #37018 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37018/consoleFull) for PR 6444 at commit [`35dad9f`](https://gith

[GitHub] spark pull request: [SPARK-8271][SQL]string function: soundex

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

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] Binary processing so...

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

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] Binary processing so...

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

[GitHub] spark pull request: [SPARK-8271][SQL]string function: soundex

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

[GitHub] spark pull request: [SPARK-8969][SQL] move type-check from BinaryA...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7338#issuecomment-120244299 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 n

[GitHub] spark pull request: [SPARK-8969][SQL] move type-check from BinaryA...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7338#issuecomment-120244247 [Test build #37003 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37003/console) for PR 7338 at commit [`4b92e48`](https://github.

[GitHub] spark pull request: [SPARK-8884] [MLlib] [WIP] 1-sample Anderson-D...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7278#issuecomment-120243436 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 project

[GitHub] spark pull request: [SPARK-8884] [MLlib] [WIP] 1-sample Anderson-D...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7278#issuecomment-120243289 [Test build #37014 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37014/console) for PR 7278 at commit [`f9778e0`](https://github.

[GitHub] spark pull request: [SPARK-6284][MESOS] Add mesos role, principal ...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4960#issuecomment-120238252 [Test build #37016 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37016/consoleFull) for PR 4960 at commit [`aa484c6`](https://gith

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] [WIP] Binary process...

2015-07-09 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/6444#issuecomment-120237679 @davies, there's one minor change in `SparkPlanTest` that I want to partially undo: the sorting prior to output comparison should continue to be opt-out, not opt-in, s

[GitHub] spark pull request: [SPARK-6284][MESOS] Add mesos role, principal ...

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

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] [WIP] Binary process...

2015-07-09 Thread JoshRosen
Github user JoshRosen commented on a diff in the pull request: https://github.com/apache/spark/pull/6444#discussion_r34329354 --- Diff: core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeSorterSpillWriter.java --- @@ -0,0 +1,143 @@ +/* + * Licensed to th

[GitHub] spark pull request: [SPARK-6284][MESOS] Add mesos role, principal ...

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

[GitHub] spark pull request: [SPARK-6284][MESOS] Add mesos role, principal ...

2015-07-09 Thread tnachen
Github user tnachen commented on the pull request: https://github.com/apache/spark/pull/4960#issuecomment-120237364 retest this please. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fe

[GitHub] spark pull request: [SPARK-8221][SQL]Add pmod function

2015-07-09 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/6783#discussion_r34329090 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala --- @@ -439,6 +439,12 @@ object HiveTypeCoercion {

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] [WIP] Binary process...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6444#issuecomment-120234581 [Test build #37015 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37015/consoleFull) for PR 6444 at commit [`5135200`](https://gith

[GitHub] spark pull request: [SPARK-8221][SQL]Add pmod function

2015-07-09 Thread zhichao-li
Github user zhichao-li commented on a diff in the pull request: https://github.com/apache/spark/pull/6783#discussion_r34328802 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/HiveTypeCoercion.scala --- @@ -439,6 +439,12 @@ object HiveTypeCoercion {

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] [WIP] Binary process...

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

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] [WIP] Binary process...

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

[GitHub] spark pull request: [CORE][Minor]change the log level to info

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7340#issuecomment-120234073 [Test build #37013 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37013/consoleFull) for PR 7340 at commit [`59658cf`](https://gith

[GitHub] spark pull request: [SPARK-8851][YARN] In Yarn client mode, Client...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7255#issuecomment-120233904 [Test build #37000 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37000/console) for PR 7255 at commit [`e13aab1`](https://github.

[GitHub] spark pull request: [SPARK-8851][YARN] In Yarn client mode, Client...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7255#issuecomment-120234002 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 project

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] [WIP] Binary process...

2015-07-09 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/6444#issuecomment-120233865 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 h

[GitHub] spark pull request: [SPARK-8884] [MLlib] [WIP] 1-sample Anderson-D...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7278#issuecomment-120233856 [Test build #37014 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37014/consoleFull) for PR 7278 at commit [`f9778e0`](https://gith

[GitHub] spark pull request: [SPARK-8884] [MLlib] [WIP] 1-sample Anderson-D...

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

[GitHub] spark pull request: [SPARK-8884] [MLlib] [WIP] 1-sample Anderson-D...

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

[GitHub] spark pull request: [CORE][Minor]change the log level to info

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

[GitHub] spark pull request: [SPARK-8969][SQL] move type-check from BinaryA...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7338#issuecomment-120233717 [Test build #37006 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37006/console) for PR 7338 at commit [`d41e80f`](https://github.

[GitHub] spark pull request: [SPARK-8969][SQL] move type-check from BinaryA...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7338#issuecomment-120233735 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 project

[GitHub] spark pull request: [CORE][Minor]change the log level to info

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

[GitHub] spark pull request: [SPARK-8884] [MLlib] [WIP] 1-sample Anderson-D...

2015-07-09 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/7278#issuecomment-120233375 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 enab

[GitHub] spark pull request: [CORE][Minor]change the log level to info

2015-07-09 Thread chenghao-intel
GitHub user chenghao-intel opened a pull request: https://github.com/apache/spark/pull/7340 [CORE][Minor]change the log level to info Too many logs even when set the log level to warning. You can merge this pull request into a Git repository by running: $ git pull https://githu

[GitHub] spark pull request: [SPARK-8675] Executors created by LocalBackend...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7091#issuecomment-120232978 [Test build #1034 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1034/console) for PR 7091 at commit [`365838f`](https://github

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6851#issuecomment-120232832 [Test build #37012 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37012/consoleFull) for PR 6851 at commit [`7a6bdbb`](https://gith

[GitHub] spark pull request: [SPARK-8221][SQL]Add pmod function

2015-07-09 Thread zhichao-li
Github user zhichao-li commented on a diff in the pull request: https://github.com/apache/spark/pull/6783#discussion_r34328421 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ArithmeticExpressionSuite.scala --- @@ -158,4 +157,18 @@ class ArithmeticE

[GitHub] spark pull request: [SPARK-8484] [ML]. Added TrainValidationSplit ...

2015-07-09 Thread feynmanliang
Github user feynmanliang commented on the pull request: https://github.com/apache/spark/pull/7337#issuecomment-120232530 Actually, is there much difference between this and `CrossValidator` other than being able to specify `trainRatio`? There seems to be a lot of code duplication betw

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

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

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

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

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] [WIP] Binary process...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6444#issuecomment-120231828 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 project

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] [WIP] Binary process...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6444#issuecomment-120231777 [Test build #36998 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36998/console) for PR 6444 at commit [`5135200`](https://github.

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

2015-07-09 Thread adrian-wang
Github user adrian-wang commented on the pull request: https://github.com/apache/spark/pull/6851#issuecomment-120230751 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 thi

[GitHub] spark pull request: [SPARK-8484] [ML]. Added TrainValidationSplit ...

2015-07-09 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7337#discussion_r34328160 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tuning/TrainValidationSplit.scala --- @@ -0,0 +1,178 @@ +/* + * Licensed to the Apache Softwar

[GitHub] spark pull request: [SPARK-8968] [SQL] shuffled by the partition c...

2015-07-09 Thread scwf
Github user scwf commented on the pull request: https://github.com/apache/spark/pull/7336#issuecomment-120229996 >This patch we shuffle data by the partition columns firstly so that each partition will have ony one partition file and this also reduce the gc overhead. @lianchen

[GitHub] spark pull request: [SPARK-8484] [ML]. Added TrainValidationSplit ...

2015-07-09 Thread feynmanliang
Github user feynmanliang commented on the pull request: https://github.com/apache/spark/pull/7337#issuecomment-120229395 Done for now. This is in very good shape. --- 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 pr

[GitHub] spark pull request: [SPARK-8636][SQL] Fix equalNullSafe comparison

2015-07-09 Thread vinodkc
Github user vinodkc commented on a diff in the pull request: https://github.com/apache/spark/pull/7040#discussion_r34328058 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/conditionals.scala --- @@ -296,9 +295,7 @@ case class CaseKeyWhen(key: Expres

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6851#issuecomment-120227701 [Test build #37011 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37011/consoleFull) for PR 6851 at commit [`7a6bdbb`](https://gith

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

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

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6851#issuecomment-120227708 [Test build #37011 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37011/console) for PR 6851 at commit [`7a6bdbb`](https://github.

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6851#issuecomment-120227720 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 project

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

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

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

2015-07-09 Thread adrian-wang
Github user adrian-wang commented on the pull request: https://github.com/apache/spark/pull/6851#issuecomment-120227302 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 thi

[GitHub] spark pull request: [SPARK-8593] [Core]: History Server: some atte...

2015-07-09 Thread rekhajoshm
Github user rekhajoshm commented on the pull request: https://github.com/apache/spark/pull/7253#issuecomment-120227192 imo that would not be correct @vanzin for the case when a1 is completed, and a2 is incomplete if a1 start times before a2, doing descending start time check would imp

[GitHub] spark pull request: [SPARK-8970][SQL] remove unnecessary abstracti...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7339#issuecomment-120227206 [Test build #37010 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37010/consoleFull) for PR 7339 at commit [`394d3ac`](https://gith

[GitHub] spark pull request: [SPARK-8970][SQL] remove unnecessary abstracti...

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

[GitHub] spark pull request: [SPARK-8970][SQL] remove unnecessary abstracti...

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

[GitHub] spark pull request: [SPARK-8675] Executors created by LocalBackend...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/7091#issuecomment-120226921 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 project

[GitHub] spark pull request: [SPARK-8675] Executors created by LocalBackend...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/7091#issuecomment-120226817 [Test build #36996 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36996/console) for PR 7091 at commit [`365838f`](https://github.

[GitHub] spark pull request: [SPARK-4072][Core]Display Streaming blocks in ...

2015-07-09 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/6672#issuecomment-120226827 retest this please --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fe

[GitHub] spark pull request: [SPARK-8970][SQL] remove unnecessary abstracti...

2015-07-09 Thread cloud-fan
GitHub user cloud-fan opened a pull request: https://github.com/apache/spark/pull/7339 [SPARK-8970][SQL] remove unnecessary abstraction for ExtractValue You can merge this pull request into a Git repository by running: $ git pull https://github.com/cloud-fan/spark minor Alter

[GitHub] spark pull request: [SPARK-8484] [ML]. Added TrainValidationSplit ...

2015-07-09 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7337#discussion_r34327741 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tuning/TrainValidationSplit.scala --- @@ -0,0 +1,178 @@ +/* + * Licensed to the Apache Softwar

[GitHub] spark pull request: [SPARK-8484] [ML]. Added TrainValidationSplit ...

2015-07-09 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7337#discussion_r34327673 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tuning/TrainValidationSplit.scala --- @@ -0,0 +1,178 @@ +/* + * Licensed to the Apache Softwar

[GitHub] spark pull request: [SPARK-8484] [ML]. Added TrainValidationSplit ...

2015-07-09 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7337#discussion_r34327642 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tuning/TrainValidationSplit.scala --- @@ -0,0 +1,178 @@ +/* + * Licensed to the Apache Softwar

[GitHub] spark pull request: [SPARK-8484] [ML]. Added TrainValidationSplit ...

2015-07-09 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7337#discussion_r34327625 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tuning/TrainValidationSplit.scala --- @@ -0,0 +1,178 @@ +/* + * Licensed to the Apache Softwar

[GitHub] spark pull request: [SPARK-8484] [ML]. Added TrainValidationSplit ...

2015-07-09 Thread feynmanliang
Github user feynmanliang commented on a diff in the pull request: https://github.com/apache/spark/pull/7337#discussion_r34327593 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tuning/TrainValidationSplit.scala --- @@ -0,0 +1,178 @@ +/* + * Licensed to the Apache Softwar

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6851#issuecomment-120224475 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 project

[GitHub] spark pull request: [SPARK-8203] [SPARK-8204] [SQL] conditional fu...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6851#issuecomment-120224398 [Test build #37001 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37001/console) for PR 6851 at commit [`7a6bdbb`](https://github.

[GitHub] spark pull request: [SPARK-7806][EC2] Fixes that allow the spark_e...

2015-07-09 Thread meawoppl
Github user meawoppl commented on the pull request: https://github.com/apache/spark/pull/6336#issuecomment-120223987 Should we chime in on that thread? --Matthew Goodman = Check Out My Website: http://craneium.net Find me on LinkedIn: http:/

[GitHub] spark pull request: [SPARK-7806][EC2] Fixes that allow the spark_e...

2015-07-09 Thread meawoppl
Github user meawoppl commented on the pull request: https://github.com/apache/spark/pull/6336#issuecomment-120223770 I dosen't appear there was ever a consensus reached there. My major complaint in this all is that I actually had to sit down an draw a giant flow chart of how al

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] [WIP] Binary process...

2015-07-09 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/6444#issuecomment-120223080 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 enabled an

[GitHub] spark pull request: [SPARK-7786][STREAMING] Allow StreamingListene...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6380#issuecomment-120222961 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 project

[GitHub] spark pull request: [SPARK-7786][STREAMING] Allow StreamingListene...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6380#issuecomment-120222941 [Test build #36995 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36995/console) for PR 6380 at commit [`25d`](https://github.

[GitHub] spark pull request: [SPARK-8743] [Streaming]: Deregister Codahale ...

2015-07-09 Thread nssalian
Github user nssalian commented on a diff in the pull request: https://github.com/apache/spark/pull/7250#discussion_r34327159 --- Diff: streaming/src/main/scala/org/apache/spark/streaming/StreamingContext.scala --- @@ -581,6 +579,9 @@ class StreamingContext private[streaming] (

[GitHub] spark pull request: [SPARK-8271][SQL]string function: soundex

2015-07-09 Thread tarekauel
Github user tarekauel commented on a diff in the pull request: https://github.com/apache/spark/pull/7115#discussion_r34327077 --- Diff: unsafe/src/main/java/org/apache/spark/unsafe/types/UTF8String.java --- @@ -47,11 +47,15 @@ 5, 5, 5, 5, 6, 6, 6, 6}; +

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] [WIP] Binary process...

2015-07-09 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6444#issuecomment-120222114 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 project

[GitHub] spark pull request: [SPARK-7078] [SPARK-7079] [WIP] Binary process...

2015-07-09 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/6444#issuecomment-120222067 [Test build #36992 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/36992/console) for PR 6444 at commit [`2f48777`](https://github.

  1   2   3   4   5   6   7   8   9   10   >