[GitHub] spark pull request: [SPARK-14514][DOC] Add python example for Vect...

2016-04-26 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/12282#issuecomment-214886579 LGTM pending an updated run of the tests I agree we can update this later after Attributes are exposed in Python. We want to re-evaluate the

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

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

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-04-26 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-214898708 LGTM. Thanks @olarayej - I just had a couple of minor comments about using `SparkDataFrame`. Other than that this looks good to merge --- If your project is set up

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

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

[GitHub] spark pull request: [SPARK-14929][SQL] Disable vectorized map for ...

2016-04-26 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12710#issuecomment-214898824 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

[GitHub] spark pull request: [SPARK-14930][SPARK-13693] Fix race condition ...

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

[GitHub] spark pull request: [SPARK-14930][SPARK-13693] Fix race condition ...

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

[GitHub] spark pull request: [SPARK-14930][SPARK-13693] Fix race condition ...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12712#issuecomment-214903401 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-14930][SPARK-13693] Fix race condition ...

2016-04-26 Thread JoshRosen
Github user JoshRosen commented on the pull request: https://github.com/apache/spark/pull/12712#issuecomment-214903452 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-13734][SPARKR] Added histogram function

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-214907846 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-13734][SPARKR] Added histogram function

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

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-04-26 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-214907828 LGTM. Merging this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

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

[GitHub] spark pull request: [SPARK-14937][ML][Document]spark.ml LogisticRe...

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

[GitHub] spark pull request: [SPARK-14937][ML][Document]spark.ml LogisticRe...

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

[GitHub] spark pull request: [SPARK-14937][ML][Document]spark.ml LogisticRe...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12717#issuecomment-214918989 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-14940][SQL] Move ExternalCatalog to own...

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

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

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

[GitHub] spark pull request: [SPARK-14937][ML][Document]spark.ml LogisticRe...

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

[GitHub] spark pull request: [SPARK-14937][ML][Document]spark.ml LogisticRe...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12717#issuecomment-214921623 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-14514][DOC] Add python example for Vect...

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

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

2016-04-26 Thread thunterdb
Github user thunterdb commented on the pull request: https://github.com/apache/spark/pull/12560#issuecomment-214890157 @wangmiao1981 sorry for the delay. I agree with @MLnick that the developer API of ALS is already brittle enough as it stands. One way to still do it is to add one

[GitHub] spark pull request: [SPARK-14514][DOC] Add python example for Vect...

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

[GitHub] spark pull request: [SPARK-14514][DOC] Add python example for Vect...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12282#issuecomment-214890032 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-14706][ML][PySpark] Python ML persisten...

2016-04-26 Thread yinxusen
Github user yinxusen commented on the pull request: https://github.com/apache/spark/pull/12604#issuecomment-214890020 I'll do it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-04-26 Thread olarayej
Github user olarayej commented on a diff in the pull request: https://github.com/apache/spark/pull/11569#discussion_r61168496 --- Diff: R/pkg/R/DataFrame.R --- @@ -2465,6 +2465,110 @@ setMethod("drop", base::drop(x) }) +#' This function

[GitHub] spark pull request: [SPARK-13477][SQL][WIP] Expose new user-facing...

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

[GitHub] spark pull request: [SPARK-14928] [SQL] support substitution in SE...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12709#issuecomment-214894951 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-14514][DOC] Add python example for Vect...

2016-04-26 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/12282#issuecomment-214895024 Merging with master Thanks @zhengruifeng ! --- 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-13734][SPARKR] Added histogram function

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-214894940 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-14928] [SQL] support substitution in SE...

2016-04-26 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12709#issuecomment-214895172 Shouldn't this already work in the parser? --- 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-12154] Upgrade to Jersey 2

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

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

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

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

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

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12689#issuecomment-214911921 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-14829][MLLIB] Deprecate GLM APIs using ...

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

[GitHub] spark pull request: [SPARK-14445][SQL] Supports native execution o...

2016-04-26 Thread dilipbiswal
Github user dilipbiswal commented on a diff in the pull request: https://github.com/apache/spark/pull/12703#discussion_r61179510 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/commands.scala --- @@ -112,3 +116,107 @@ case class ExplainCommand(

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

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

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

2016-04-26 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-214890244 @shivaram @felixcheung Looks like the version of lint-r running on the build server is different than the one on Spark's Github. Even though lint-r passes on my

[GitHub] spark pull request: [SPARK-12301][ML] Made all tree and ensemble c...

2016-04-26 Thread thunterdb
Github user thunterdb commented on the pull request: https://github.com/apache/spark/pull/12711#issuecomment-214895988 @jkbradley 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

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12689#issuecomment-214898218 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-14706][ML][PySpark] Python ML persisten...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12604#issuecomment-214898299 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-13734][SPARKR] Added histogram function

2016-04-26 Thread olarayej
Github user olarayej commented on the pull request: https://github.com/apache/spark/pull/11569#issuecomment-214898243 @shivaram @felixcheung I have addressed all your comments. Anything else? Or shall we merge? Thanks! --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-14706][ML][PySpark] Python ML persisten...

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

[GitHub] spark pull request: [SPARK-14706][ML][PySpark] Python ML persisten...

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

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

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

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

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

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

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

[GitHub] spark pull request: [SPARK-14706][ML][PySpark] Python ML persisten...

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

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

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

[GitHub] spark pull request: [SPARK-14930][SPARK-13693] Fix race condition ...

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

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

2016-04-26 Thread yhuai
Github user yhuai commented on the pull request: https://github.com/apache/spark/pull/12689#issuecomment-214905568 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

[GitHub] spark pull request: [SPARK-12154] Upgrade to Jersey 2

2016-04-26 Thread mccheah
Github user mccheah commented on the pull request: https://github.com/apache/spark/pull/11223#issuecomment-214905547 Moved to #12715 --- 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-14913][SQL] Simplify configuration API

2016-04-26 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/12689#discussion_r61176814 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/SparkSession.scala --- @@ -205,7 +201,7 @@ class SparkSession private( * @group config

[GitHub] spark pull request: [SPARK-13477][SQL][WIP] Expose new user-facing...

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

[GitHub] spark pull request: [SPARK-13477][SQL][WIP] Expose new user-facing...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12713#issuecomment-214911403 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-14130] [SQL] Throw exceptions for ALTER...

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

[GitHub] spark pull request: [SPARK-14130] [SQL] Throw exceptions for ALTER...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12714#issuecomment-214914523 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-14829][MLLIB] Deprecate GLM APIs using ...

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

[GitHub] spark pull request: [SPARK-14445][SQL] Supports native execution o...

2016-04-26 Thread dilipbiswal
Github user dilipbiswal commented on a diff in the pull request: https://github.com/apache/spark/pull/12703#discussion_r61180561 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/commands.scala --- @@ -112,3 +116,107 @@ case class ExplainCommand(

[GitHub] spark pull request: [SPARK-11940][PYSPARK][ML] Python API for ml.c...

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

[GitHub] spark pull request: [SPARK-14939][SQL] Eliminate No-op Alias SortO...

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

[GitHub] spark pull request: [SPARK-14706][ML][PySpark] Python ML persisten...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12604#issuecomment-214918091 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-11940][PYSPARK][ML] Python API for ml.c...

2016-04-26 Thread zjffdu
Github user zjffdu commented on the pull request: https://github.com/apache/spark/pull/10242#issuecomment-214918065 @jkbradley I made some update based your comments before, but don't have time to implement the model persistence feature. Please take over this PR. --- If your project

[GitHub] spark pull request: [SPARK-11940][PYSPARK][ML] Python API for ml.c...

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

[GitHub] spark pull request: [SPARK-14938][ML] replace rdd with dataset

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

[GitHub] spark pull request: [SPARK-14706][ML][PySpark] Python ML persisten...

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

[GitHub] spark pull request: [SPARK-14937][ML][Document]spark.ml LogisticRe...

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

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

2016-04-26 Thread andrewor14
GitHub user andrewor14 opened a pull request: https://github.com/apache/spark/pull/12721 [SPARK-14940][SQL] Move ExternalCatalog to own file ## What changes were proposed in this pull request? `interfaces.scala` was getting big. This just moves the biggest class in there

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

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

[GitHub] spark pull request: [SPARK-14930][SPARK-13693 Fix race condition i...

2016-04-26 Thread JoshRosen
GitHub user JoshRosen opened a pull request: https://github.com/apache/spark/pull/12712 [SPARK-14930][SPARK-13693 Fix race condition in CheckpointWriter.stop() CheckpointWriter.stop() is prone to a race condition: if one thread calls `stop()` right as a checkpoint write task begins

[GitHub] spark pull request: [SPARK-14929][SQL] Disable vectorized map for ...

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

[GitHub] spark pull request: [SPARK-14929][SQL] Disable vectorized map for ...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12710#issuecomment-214897701 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-12154] Upgrade to Jersey 2

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12715#issuecomment-214902311 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-12154] Upgrade to Jersey 2

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

[GitHub] spark pull request: [SPARK-12154] Upgrade to Jersey 2

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

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

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

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

2016-04-26 Thread andrewor14
Github user andrewor14 commented on a diff in the pull request: https://github.com/apache/spark/pull/12689#discussion_r61177026 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLContextSuite.scala --- @@ -85,27 +85,4 @@ class SQLContextSuite extends SparkFunSuite with

[GitHub] spark pull request: [Spark-14933][SQL] Failed to create view out o...

2016-04-26 Thread xwu0226
Github user xwu0226 commented on the pull request: https://github.com/apache/spark/pull/12716#issuecomment-214909072 @liancheng I hit this issue when I tied to create a testcase for the `show create table` PR. So I created a separate PR to address this issue. Please help take a look

[GitHub] spark pull request: [SPARK-14913][SQL] Simplify configuration API

2016-04-26 Thread andrewor14
Github user andrewor14 commented on the pull request: https://github.com/apache/spark/pull/12689#issuecomment-214909100 other than the lazy initialization thing this LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [SPARK-13477][SQL][WIP] Expose new user-facing...

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

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

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

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

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

[GitHub] spark pull request: [SPARK-14571][ML]Log instrumentation in ALS

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12560#issuecomment-214913413 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-13477][SQL][WIP] Expose new user-facing...

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

[GitHub] spark pull request: [SPARK-14937][ML][Document]spark.ml LogisticRe...

2016-04-26 Thread wangmiao1981
GitHub user wangmiao1981 opened a pull request: https://github.com/apache/spark/pull/12717 [SPARK-14937][ML][Document]spark.ml LogisticRegression sqlCtx in scala is inconsistent with java and python ## What changes were proposed in this pull request? In spark.ml document, the

[GitHub] spark pull request: [SPARK-14487][SQL] User Defined Type registrat...

2016-04-26 Thread viirya
Github user viirya commented on the pull request: https://github.com/apache/spark/pull/12259#issuecomment-214916566 ping @dbtsai @mengxr Can you take a look? Any more comments or is this ready to merge now? Thanks. --- If your project is set up for it, you can reply to this email

[GitHub] spark pull request: [SPARK-14937][ML][Document]spark.ml LogisticRe...

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

[GitHub] spark pull request: [SPARK-14937][ML][Document]spark.ml LogisticRe...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12717#issuecomment-214922321 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-14940][SQL] Move ExternalCatalog to own...

2016-04-26 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12721#issuecomment-214922261 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-14940][SQL] Move ExternalCatalog to own...

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

[GitHub] spark pull request: [SPARK-14940][SQL] Move ExternalCatalog to own...

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

[GitHub] spark pull request: [SPARK-14829][MLLIB] Deprecate GLM APIs using ...

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

[GitHub] spark pull request: [SPARK-14732][ML] spark.ml GaussianMixture sho...

2016-04-26 Thread dbtsai
Github user dbtsai commented on the pull request: https://github.com/apache/spark/pull/12593#issuecomment-214923459 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

[GitHub] spark pull request: [SPARK-14732][ML] spark.ml GaussianMixture sho...

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

[GitHub] spark pull request: [SPARK-14480][SQL] Simplify CSV parsing proces...

2016-04-26 Thread HyukjinKwon
Github user HyukjinKwon commented on the pull request: https://github.com/apache/spark/pull/12268#issuecomment-214925695 @rxin No problem. Let me just rebase it if it has conflicts anyway. It is easier to track the changes. --- If your project is set up for it, you can reply to this

[GitHub] spark pull request: [SPARK-14159][ML] Fixed bug in StringIndexer +...

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

[GitHub] spark pull request: [SPARK-14585][ML][WIP] Provide accessor method...

2016-04-26 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/12420#issuecomment-214885225 By the way, we'll need to get this merged quickly for it to make 2.0. (Thanks!) --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

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

[GitHub] spark pull request: [SPARK-13734][SPARKR] Added histogram function

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

  1   2   3   4   5   6   7   8   9   10   >