[GitHub] spark pull request: [SPARK-10258] [DOCUMENTATION, ML] Adding Since...

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

[GitHub] spark pull request: [SPARK-6880][CORE] Use properties from ActiveJ...

2015-09-17 Thread squito
Github user squito commented on the pull request: https://github.com/apache/spark/pull/6291#issuecomment-141140963 hi @markhamstra I think this looks good, but after thinking about it more -- I kinda feel that this deserves its own jira. It would be helpful to users on old versions,

[GitHub] spark pull request: [SPARK-10459][SQL] Do not need to have Convert...

2015-09-17 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/8616#issuecomment-141138181 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-10459][SQL] Do not need to have Convert...

2015-09-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8616 --- 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-10258] [DOCUMENTATION, ML] Adding Since...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8505#issuecomment-141138756 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 this f

[GitHub] spark pull request: [SPARK-10258] [DOCUMENTATION, ML] Adding Since...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8505#issuecomment-141138681 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 have thi

[GitHub] spark pull request: [SPARK-10459][SQL] Do not need to have Convert...

2015-09-17 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/8616#issuecomment-141138417 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 have

[GitHub] spark pull request: [SPARK-10258] [DOCUMENTATION, ML] Adding Since...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8505#issuecomment-141137776 @martinbrown there are some merge conflicts. could you rebase master? --- 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-10077][DOCS][ML] Add package info for j...

2015-09-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8740 --- 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-10258] [DOCUMENTATION, ML] Adding Since...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8505#issuecomment-141137687 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: [SPARK-10077][DOCS][ML] Add package info for j...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8740#issuecomment-141137424 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-9642] [ML] [WIP] LinearRegression shoul...

2015-09-17 Thread rotationsymmetry
Github user rotationsymmetry commented on the pull request: https://github.com/apache/spark/pull/8631#issuecomment-141136871 retest this please. "org.apache.spark.HeartbeatReceiverSuite.reregister if heartbeat from removed executor" failed, which should be unrelated to this pa

[GitHub] spark pull request: [SPARK-10642][PySpark] Fix crash when calling ...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8796#issuecomment-141134569 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-10642][PySpark] Fix crash when calling ...

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

[GitHub] spark pull request: [SPARK-10642][PySpark] Fix crash when calling ...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8796#issuecomment-141134539 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-10642][PySpark] Fix crash when calling ...

2015-09-17 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/8796#issuecomment-141133821 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 fea

[GitHub] spark pull request: [SPARK-10540] Fixes flaky all-data-type test

2015-09-17 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/8768#issuecomment-141133637 Can we let every test run more times (50 or 100 times) before we merge it? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-10642][PySpark] Fix crash when calling ...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8796#issuecomment-141133177 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-10642][PySpark] Fix crash when calling ...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8796#issuecomment-141133179 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42

[GitHub] spark pull request: [SPARK-10274][MLlib] Add @since annotation to ...

2015-09-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8665 --- 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-10284][ML][PySpark][Docs] Add @since an...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8694#issuecomment-141129926 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 have

[GitHub] spark pull request: [SPARK-10283][ML][PySpark][Docs] Add @since an...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8693#issuecomment-141129971 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 have

[GitHub] spark pull request: [SPARK-10282][ML][PySpark][Docs] Add @since an...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8692#issuecomment-141130127 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 have

[GitHub] spark pull request: [SPARK-10278][MLlib][PySpark] Add @since annot...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8685#issuecomment-141130051 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 have

[GitHub] spark pull request: [SPARK-10282][ML][PySpark][Docs] Add @since an...

2015-09-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8692 --- 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-10281][ML][PySpark][Docs] Add @since an...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8691#issuecomment-141130025 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 have

[GitHub] spark pull request: [SPARK-10274][MLlib] Add @since annotation to ...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8665#issuecomment-141130107 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 have

[GitHub] spark pull request: [SPARK-10279][MLlib][PySpark][Docs] Add @since...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8689#issuecomment-141130078 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 have

[GitHub] spark pull request: [SPARK-10279][MLlib][PySpark][Docs] Add @since...

2015-09-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8689 --- 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-10278][MLlib][PySpark] Add @since annot...

2015-09-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8685 --- 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-10281][ML][PySpark][Docs] Add @since an...

2015-09-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8691 --- 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-10642][PySpark] Fix crash when calling ...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8796#issuecomment-141128478 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-10642][PySpark] Fix crash when calling ...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8796#issuecomment-141128526 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-10283][ML][PySpark][Docs] Add @since an...

2015-09-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8693 --- 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-10284][ML][PySpark][Docs] Add @since an...

2015-09-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8694 --- 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-10394] [ML] Make GBTParams use shared s...

2015-09-17 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8552#discussion_r39761223 --- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala --- @@ -393,11 +383,19 @@ private[ml] trait GBTParams extends TreeEnsembleParams with

[GitHub] spark pull request: [SPARK-10280][MLlib][PySpark][Docs] Add @since...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8690#issuecomment-141126346 @noel-smith New PRs should come along with `@since`. I forgot that for MLPC Python API. @yu-iskw Could you rebase master? --- If your project is set up for it, you can r

[GitHub] spark pull request: [SPARK-10277][MLlib][PySpark] Add @since annot...

2015-09-17 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8684#issuecomment-141125190 @yu-iskw Please keep the content minimal. This PR shouldn't contain doc changes. Otherwise, it would slow down the code review process. For example, there are issues with

[GitHub] spark pull request: [SPARK-8632] [SQL] [PYSPARK] Poor Python UDF p...

2015-09-17 Thread justinuang
Github user justinuang commented on the pull request: https://github.com/apache/spark/pull/8662#issuecomment-141121225 @rxin what do you mean by local iterators =) I feel like i'm missing some context that you guys have --- If your project is set up for it, you can reply to this emai

[GitHub] spark pull request: [SPARK-10474][SQL][WIP]Avoid eat out all of th...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8798#issuecomment-141121402 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-10474][SQL][WIP]Avoid eat out all of th...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8798#issuecomment-141121406 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42

[GitHub] spark pull request: [SPARK-10474][SQL][WIP]Avoid eat out all of th...

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

[GitHub] spark pull request: [SPARK-10515] When killing executor, the pendi...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8668#issuecomment-141119832 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-10515] When killing executor, the pendi...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8668#issuecomment-141119835 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42

[GitHub] spark pull request: [SPARK-10515] When killing executor, the pendi...

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

[GitHub] spark pull request: [SPARK-10394] [ML] Make GBTParams use shared s...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8552#issuecomment-141119101 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42

[GitHub] spark pull request: [SPARK-10394] [ML] Make GBTParams use shared s...

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

[GitHub] spark pull request: [SPARK-10394] [ML] Make GBTParams use shared s...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8552#issuecomment-141119094 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-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39757885 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala --- @@ -0,0 +1,226 @@ +/* + * Licensed to the Apache Software Foundation (AS

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

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

[GitHub] spark pull request: [SPARK-8632] [SQL] [PYSPARK] Poor Python UDF p...

2015-09-17 Thread justinuang
Github user justinuang commented on the pull request: https://github.com/apache/spark/pull/8662#issuecomment-141117878 The solution with the iterator wrapper was my first approach that I prototyped (http://apache-spark-developers-list.1001551.n3.nabble.com/Python-UDF-performance-at-la

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39756753 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Inbox.scala --- @@ -0,0 +1,229 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39756653 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala --- @@ -0,0 +1,226 @@ +/* + * Licensed to the Apache Software Foundation (AS

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6457#issuecomment-141116221 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-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39756578 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala --- @@ -0,0 +1,226 @@ +/* + * Licensed to the Apache Software Foundation (AS

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6457#issuecomment-141116266 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-10471] [CORE] [MESOS] prevent getting o...

2015-09-17 Thread tnachen
Github user tnachen commented on the pull request: https://github.com/apache/spark/pull/8639#issuecomment-141115147 There is a big HTML table in the bottom of this file, can you also add it to that list? On Thu, Sep 17, 2015 at 4:55 AM, Akash Mishra wrote: > @

[GitHub] spark pull request: Spark-10625: Spark SQL JDBC read/write is unab...

2015-09-17 Thread tribbloid
Github user tribbloid commented on the pull request: https://github.com/apache/spark/pull/8785#issuecomment-141114437 Does it look any better now? --- 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-4131][SQL] support writing data into th...

2015-09-17 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/4380#issuecomment-141113842 @litao-buptsse Since it is a new feature, I think we will not get it in branch 1.5. But, we can target 1.6. --- If your project is set up for it, you can reply to this em

[GitHub] spark pull request: [SPARK-9522][SQL] SparkSubmit process can not ...

2015-09-17 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/7853#discussion_r39754972 --- Diff: sql/hive-thriftserver/src/main/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2.scala --- @@ -93,6 +93,11 @@ object HiveThriftServer2 e

[GitHub] spark pull request: [SPARK-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8611#issuecomment-141110526 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-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8611#issuecomment-141110530 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42

[GitHub] spark pull request: [SPARK-8518] [ML] Log-linear models for surviv...

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

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6457#issuecomment-141106261 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-6028][Core]A new RPC implemetation base...

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

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39752517 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Inbox.scala --- @@ -0,0 +1,229 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

[GitHub] spark pull request: [SPARK-10577] [PySpark] DataFrame hint for bro...

2015-09-17 Thread Jianfeng-chs
Github user Jianfeng-chs commented on the pull request: https://github.com/apache/spark/pull/8777#issuecomment-141106831 Since messed commits, closing it --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project doe

[GitHub] spark pull request: [SPARK-10577] [PySpark] DataFrame hint for bro...

2015-09-17 Thread Jianfeng-chs
Github user Jianfeng-chs closed the pull request at: https://github.com/apache/spark/pull/8777 --- 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 i

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39752635 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/NettyRpcAddress.scala --- @@ -0,0 +1,56 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39752618 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Inbox.scala --- @@ -0,0 +1,229 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/6457#issuecomment-141106342 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-10657] Remove SCP-based Jenkins log arc...

2015-09-17 Thread shaneknapp
Github user shaneknapp commented on the pull request: https://github.com/apache/spark/pull/8793#issuecomment-141105891 LGTM! make it so! --- 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-10394] [ML] Make GBTParams use shared s...

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

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39752404 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Inbox.scala --- @@ -0,0 +1,229 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39752100 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Inbox.scala --- @@ -0,0 +1,229 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39752148 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Inbox.scala --- @@ -0,0 +1,229 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39752031 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala --- @@ -0,0 +1,226 @@ +/* + * Licensed to the Apache Software Foundation (AS

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39752120 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Inbox.scala --- @@ -0,0 +1,229 @@ +/* + * Licensed to the Apache Software Foundation (ASF) und

[GitHub] spark pull request: [SPARK-6028][Core]A new RPC implemetation base...

2015-09-17 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/6457#discussion_r39752048 --- Diff: core/src/main/scala/org/apache/spark/rpc/netty/Dispatcher.scala --- @@ -0,0 +1,226 @@ +/* + * Licensed to the Apache Software Foundation (AS

[GitHub] spark pull request: [SPARK-10394] [ML] Make GBTParams use shared s...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8552#issuecomment-141104991 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-10394] [ML] Make GBTParams use shared s...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8552#issuecomment-141104942 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-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/8611#issuecomment-141102645 @mengxr @rotationsymmetry Thanks for your comments. I have updated the PR, Would you mind to make another pass? --- If your project is set up for it, you can reply t

[GitHub] spark pull request: [SPARK-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/8611#discussion_r39750273 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/AFTSurvivalRegression.scala --- @@ -0,0 +1,400 @@ +/* + * Licensed to the Apache Sof

[GitHub] spark pull request: [SPARK-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/8611#discussion_r39750238 --- Diff: mllib/src/test/scala/org/apache/spark/ml/regression/AFTSurvivalRegressionSuite.scala --- @@ -0,0 +1,283 @@ +/* + * Licensed to the Apach

[GitHub] spark pull request: [SPARK-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/8611#discussion_r39750108 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/AFTSurvivalRegression.scala --- @@ -0,0 +1,400 @@ +/* + * Licensed to the Apache Sof

[GitHub] spark pull request: [SPARK-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/8611#discussion_r39749679 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/AFTSurvivalRegression.scala --- @@ -0,0 +1,400 @@ +/* + * Licensed to the Apache Sof

[GitHub] spark pull request: [SPARK-10506][SQL]move close() to finally code...

2015-09-17 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8664#issuecomment-141100130 [Test build #1770 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1770/console) for PR 8664 at commit [`4b91eac`](https://github

[GitHub] spark pull request: [SPARK-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/8611#discussion_r39749414 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/AFTSurvivalRegression.scala --- @@ -0,0 +1,400 @@ +/* + * Licensed to the Apache Sof

[GitHub] spark pull request: [SPARK-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/8611#discussion_r39749007 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/AFTSurvivalRegression.scala --- @@ -0,0 +1,400 @@ +/* + * Licensed to the Apache Sof

[GitHub] spark pull request: [SPARK-8518] [ML] Log-linear models for surviv...

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

[GitHub] spark pull request: [SPARK-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8611#issuecomment-141094736 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-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8611#issuecomment-141094758 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-8518] [ML] Log-linear models for surviv...

2015-09-17 Thread yanboliang
Github user yanboliang commented on a diff in the pull request: https://github.com/apache/spark/pull/8611#discussion_r39747513 --- Diff: mllib/src/main/scala/org/apache/spark/ml/regression/AFTSurvivalRegression.scala --- @@ -0,0 +1,400 @@ +/* + * Licensed to the Apache Sof

[GitHub] spark pull request: Upgrade the json4s version

2015-09-17 Thread wb14123
Github user wb14123 commented on the pull request: https://github.com/apache/spark/pull/5691#issuecomment-141092917 "userClassPathFirst" is marked "Experimental" in the doc for now. And it will use other library versions in my project like akka-slf4j which is incompatible with other a

[GitHub] spark pull request: [SPARK-10474][SQL][WIP]Avoid eat out all of th...

2015-09-17 Thread chenghao-intel
Github user chenghao-intel commented on a diff in the pull request: https://github.com/apache/spark/pull/8798#discussion_r39745880 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/TungstenAggregationIterator.scala --- @@ -375,7 +377,11 @@ class TungstenAg

[GitHub] spark pull request: Upgrade the json4s version

2015-09-17 Thread srowen
Github user srowen commented on the pull request: https://github.com/apache/spark/pull/5691#issuecomment-141089285 @wb14123 This issue concerns updating Spark's json4s. If you want to pursue that, you can open a PR, but the question will be the same: does it cause any compatibility pr

[GitHub] spark pull request: [SPARK-10474][SQL][WIP]Avoid eat out all of th...

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

[GitHub] spark pull request: Upgrade the json4s version

2015-09-17 Thread wb14123
Github user wb14123 commented on the pull request: https://github.com/apache/spark/pull/5691#issuecomment-141085138 @srowen Sorry, I don't understand what do you mean. The discussion here seems that the developer don't follow up. But the issue is still here and I think it is important

[GitHub] spark pull request: [SPARK-10474][SQL][WIP]Avoid eat out all of th...

2015-09-17 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8798#issuecomment-141084687 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-10474][SQL][WIP]Avoid eat out all of th...

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

<    1   2   3   4   5   6   7   >