[GitHub] spark pull request: [SPARK-15145][ML]:spark.ml binary classificati...

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

[GitHub] spark pull request: [Spark-15051] [SQL] Create a TypedColumn alias...

2016-05-04 Thread kevinyu98
Github user kevinyu98 commented on a diff in the pull request: https://github.com/apache/spark/pull/12893#discussion_r62150990 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/Column.scala --- @@ -68,6 +68,25 @@ class TypedColumn[-T, U]( } new TypedColumn[T,

[GitHub] spark pull request: [SPARK-15143][SPARK-15144][SQL] Add CSV tests ...

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

[GitHub] spark pull request: [SPARK-15143][SPARK-15144][SQL] Add CSV tests ...

2016-05-04 Thread HyukjinKwon
GitHub user HyukjinKwon opened a pull request: https://github.com/apache/spark/pull/12921 [SPARK-15143][SPARK-15144][SQL] Add CSV tests with HadoopFsRelationTest and support for nullValue for other types ## What changes were proposed in this pull request? Currently,

[GitHub] spark pull request: [SPARK-15145][ML]:spark.ml binary classificati...

2016-05-04 Thread wangmiao1981
GitHub user wangmiao1981 opened a pull request: https://github.com/apache/spark/pull/12922 [SPARK-15145][ML]:spark.ml binary classification should include accuracy ## What changes were proposed in this pull request? Add accuracy into binary classification metrics.

[GitHub] spark pull request: [SPARK-15141][DOC] Add python example for OneV...

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

[GitHub] spark pull request: [SPARK-14915] [CORE] Don't re-queue a task if ...

2016-05-04 Thread jasonmoore2k
Github user jasonmoore2k commented on the pull request: https://github.com/apache/spark/pull/12751#issuecomment-217081200 Done! Thanks for the review. --- 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-14772][ML,PySpark]Python ML Params.copy...

2016-05-04 Thread hujy
Github user hujy commented on the pull request: https://github.com/apache/spark/pull/12888#issuecomment-217081026 @holdenk, I'm checking with @jkbradley about "respect the difference between defaultParamMap and paramMap" in the requirement. The new input parameter is optional and

[GitHub] spark pull request: [SPARK-15108] [SQL] Describe Permanent UDTF

2016-05-04 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/12885#discussion_r62150647 --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/command/functions.scala --- @@ -129,7 +129,16 @@ case class DescribeFunction(

[GitHub] spark pull request: [SPARK-14897] [SQL] [WIP] upgrade to jetty 9.2...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12916#issuecomment-217078875 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-14897] [SQL] [WIP] upgrade to jetty 9.2...

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

[GitHub] spark pull request: [SPARK-14897] [SQL] [WIP] upgrade to jetty 9.2...

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

[GitHub] spark pull request: [SPARK-15093][SQL] create/delete/rename direct...

2016-05-04 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/12871#discussion_r62149583 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala --- @@ -245,7 +301,21 @@ class InMemoryCatalog extends

[GitHub] spark pull request: [SPARK-15045] [CORE] Remove dead code in TaskM...

2016-05-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12829 --- 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-15045] [CORE] Remove dead code in TaskM...

2016-05-04 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12829#issuecomment-217078394 LGTM, merging 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-15093][SQL] create/delete/rename direct...

2016-05-04 Thread dilipbiswal
Github user dilipbiswal commented on a diff in the pull request: https://github.com/apache/spark/pull/12871#discussion_r62149522 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala --- @@ -245,7 +301,21 @@ class InMemoryCatalog

[GitHub] spark pull request: [SPARK-15141][DOC] Add python example for OneV...

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

[GitHub] spark pull request: [SPARK-15141][DOC] Add python example for OneV...

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

[GitHub] spark pull request: [SPARK-15141][DOC] Add python example for OneV...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12920#issuecomment-217078369 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: [Minor][SQL][SPARK-15132] Debug log for genera...

2016-05-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12908 --- 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: [Minor][SQL][SPARK-15132] Debug log for genera...

2016-05-04 Thread rxin
Github user rxin commented on the pull request: https://github.com/apache/spark/pull/12908#issuecomment-217078212 Thanks - merging in master/branch-2.0. --- 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-15141][DOC] Add python example for OneV...

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

[GitHub] spark pull request: [SPARK-15045] [CORE] Remove dead code in TaskM...

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

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread NarineK
Github user NarineK commented on the pull request: https://github.com/apache/spark/pull/12887#issuecomment-217077799 sure! --- 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-15045] [CORE] Remove dead code in TaskM...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12829#issuecomment-217077811 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-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12887#issuecomment-217077679 @NarineK It will be great if could update that too. --- 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-15045] [CORE] Remove dead code in TaskM...

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

[GitHub] spark pull request: [SPARK-15093][SQL] create/delete/rename direct...

2016-05-04 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/12871#discussion_r62149040 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala --- @@ -245,7 +301,21 @@ class InMemoryCatalog extends

[GitHub] spark pull request: [SPARK-15141][DOC] Add python example for OneV...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12920#issuecomment-217077508 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-15141][DOC] Add python example for OneV...

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

[GitHub] spark pull request: [SPARK-15141][DOC] Add python example for OneV...

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

[GitHub] spark pull request: [SPARK-15141][DOC] Add python example for OneV...

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

[GitHub] spark pull request: [SPARK-15141][DOC] Add python example for OneV...

2016-05-04 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/12920#issuecomment-217077404 There seems no similar API to directly get `numClass` like scala `val numClasses = MetadataUtils.getNumClasses(predictionColSchema).get` --- If your project

[GitHub] spark pull request: [SPARK-15141][DOC] Add python example for OneV...

2016-05-04 Thread zhengruifeng
GitHub user zhengruifeng opened a pull request: https://github.com/apache/spark/pull/12920 [SPARK-15141][DOC] Add python example for OneVsRest ## What changes were proposed in this pull request? Add python example for OneVsRest ## How was this patch tested?

[GitHub] spark pull request: [SPARK-15093][SQL] create/delete/rename direct...

2016-05-04 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/12871#discussion_r62148788 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala --- @@ -104,6 +119,14 @@ class InMemoryCatalog extends

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread NarineK
Github user NarineK commented on the pull request: https://github.com/apache/spark/pull/12887#issuecomment-217076840 So, do you want me to update the Dataset.scala too or only the doc in R ? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] spark pull request: [SPARK-15093][SQL] create/delete/rename direct...

2016-05-04 Thread gatorsmile
Github user gatorsmile commented on a diff in the pull request: https://github.com/apache/spark/pull/12871#discussion_r62148671 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/InMemoryCatalog.scala --- @@ -85,6 +93,13 @@ class InMemoryCatalog extends

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/12887#discussion_r62148621 --- Diff: R/pkg/R/DataFrame.R --- @@ -570,10 +570,17 @@ setMethod("unpersist", #' Repartition #' -#' Return a new SparkDataFrame that has

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12887#discussion_r62148622 --- Diff: R/pkg/R/DataFrame.R --- @@ -570,10 +570,17 @@ setMethod("unpersist", #' Repartition #' -#' Return a new SparkDataFrame that has

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/12887#discussion_r62148282 --- Diff: R/pkg/R/DataFrame.R --- @@ -570,10 +570,17 @@ setMethod("unpersist", #' Repartition #' -#' Return a new SparkDataFrame that

[GitHub] spark pull request: [SPARK-15094][SQL] Avoid extra Project when De...

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

[GitHub] spark pull request: [SPARK-15094][SQL] Avoid extra Project when De...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12898#issuecomment-217075670 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-15094][SQL] Avoid extra Project when De...

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

[GitHub] spark pull request: [SPARK-12837][CORE] reduce network IO for accu...

2016-05-04 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/12899#issuecomment-217075463 I will be great we could have a test to make sure that we always have this behavior. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [SPARK-15139][SPARK-15092][PYSPARK][ML] Pyspar...

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

[GitHub] spark pull request: [SPARK-15139][SPARK-15092][PYSPARK][ML] Pyspar...

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

[GitHub] spark pull request: [SPARK-15139][SPARK-15092][PYSPARK][ML] Pyspar...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12919#issuecomment-217075399 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-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/12887#discussion_r62148066 --- Diff: R/pkg/R/DataFrame.R --- @@ -570,10 +570,17 @@ setMethod("unpersist", #' Repartition #' -#' Return a new SparkDataFrame that has

[GitHub] spark pull request: [SPARK-14139][SQL] RowEncoder should preserve ...

2016-05-04 Thread liancheng
Github user liancheng commented on a diff in the pull request: https://github.com/apache/spark/pull/12364#discussion_r62148037 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/encoders/RowEncoder.scala --- @@ -34,9 +34,8 @@ import

[GitHub] spark pull request: [SPARK-12837][CORE] reduce network IO for accu...

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

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

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12268#issuecomment-217075071 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-12837][CORE] reduce network IO for accu...

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

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

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

[GitHub] spark pull request: [SPARK-12837][CORE] reduce network IO for accu...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12899#issuecomment-217075162 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-14480][SQL] Simplify CSV parsing proces...

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

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread NarineK
Github user NarineK commented on a diff in the pull request: https://github.com/apache/spark/pull/12887#discussion_r62147815 --- Diff: R/pkg/R/DataFrame.R --- @@ -570,10 +570,17 @@ setMethod("unpersist", #' Repartition #' -#' Return a new SparkDataFrame that has

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

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

[GitHub] spark pull request: [SPARK-15131][SQL] Shutdown StateStore managem...

2016-05-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/12905 --- 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-15139][SPARK-15092][PYSPARK][ML] Pyspar...

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

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

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

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-217074490 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-14124] [SQL] [FOLLOWUP] Implement Datab...

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

[GitHub] spark pull request: [SPARK-15139][SPARK-15092][PYSPARK][ML] Pyspar...

2016-05-04 Thread holdenk
GitHub user holdenk opened a pull request: https://github.com/apache/spark/pull/12919 [SPARK-15139][SPARK-15092][PYSPARK][ML] Pyspark TreeEnsemble missing methods ## What changes were proposed in this pull request? Add `toDebugString` and `totalNumNodes` to

[GitHub] spark pull request: [SPARK-14098][SQL] Generate Java code that get...

2016-05-04 Thread davies
Github user davies commented on the pull request: https://github.com/apache/spark/pull/11956#issuecomment-217074291 InMemoryTableScanExec is a case class, not singleton, why can we not implement these there? --- If your project is set up for it, you can reply to this email and have

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

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

[GitHub] spark pull request: [SPARK-14124] [SQL] [FOLLOWUP] Implement Datab...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12081#issuecomment-217074208 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-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread davies
Github user davies commented on a diff in the pull request: https://github.com/apache/spark/pull/12887#discussion_r62147393 --- Diff: R/pkg/R/DataFrame.R --- @@ -570,10 +570,17 @@ setMethod("unpersist", #' Repartition #' -#' Return a new SparkDataFrame that has

[GitHub] spark pull request: [SPARK-15131][SQL] Shutdown StateStore managem...

2016-05-04 Thread zsxwing
Github user zsxwing commented on the pull request: https://github.com/apache/spark/pull/12905#issuecomment-217073766 LGTM. Merging to master / 2.0 --- 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-14050] [ML] Add multiple languages supp...

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

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/spark/pull/12887#issuecomment-217072731 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

[GitHub] spark pull request: [SPARK-15137][PySpark][DOC] Fix links to sphin...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12918#issuecomment-217072638 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-15137][PySpark][DOC] Fix links to sphin...

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

[GitHub] spark pull request: [SPARK-15137][PySpark][DOC] Fix links to sphin...

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

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread shivaram
Github user shivaram commented on the pull request: https://github.com/apache/spark/pull/12887#issuecomment-217072497 LGTM. Thanks @NarineK -- @davies Can you take one final look and then merge ? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-11395][SPARKR] Support over and window ...

2016-05-04 Thread sun-rui
Github user sun-rui commented on a diff in the pull request: https://github.com/apache/spark/pull/10094#discussion_r62146426 --- Diff: R/pkg/R/window.R --- @@ -0,0 +1,92 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license

[GitHub] spark pull request: [SPARK-15137][PySpark][DOC] Fix links to sphin...

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

[GitHub] spark pull request: [SPARK-14914] Skip some test cases on Windows ...

2016-05-04 Thread taoli91
Github user taoli91 commented on a diff in the pull request: https://github.com/apache/spark/pull/12696#discussion_r62146131 --- Diff: core/src/test/scala/org/apache/spark/deploy/SparkSubmitSuite.scala --- @@ -575,8 +575,13 @@ class SparkSubmitSuite // NOTE: This is an

[GitHub] spark pull request: [SPARK-14684] [SPARK-15026] [SQL] Disallow Dro...

2016-05-04 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12801#issuecomment-217070994 Regarding SPARK-15026, my major concern is we might get JIRAs if the previous `Drop Partition` command only drops a subset of partitions. If users issue the same

[GitHub] spark pull request: [SPARK-15137][PySpark][DOC] Fix links to sphin...

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

[GitHub] spark pull request: [SPARK-15137][PySpark][DOC] Fix links to sphin...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12918#issuecomment-217070776 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-15137][PySpark][DOC] Fix links to sphin...

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

[GitHub] spark pull request: [SPARK-15137][PySpark][DOC] Fix links to sphin...

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

[GitHub] spark pull request: [SPARK-14634][ML] Add BisectingKMeansSummary

2016-05-04 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/12394#issuecomment-217070537 cc @jkbradley @mengxr Summary for BisectingKMeans is still missing now --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [SPARK-15094][SQL] Avoid extra Project when De...

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

[GitHub] spark pull request: [SPARK-15137][PySpark][DOC] Fix links to sphin...

2016-05-04 Thread holdenk
GitHub user holdenk opened a pull request: https://github.com/apache/spark/pull/12918 [SPARK-15137][PySpark][DOC] Fix links to sphinx style and add a default param doc note ## What changes were proposed in this pull request? PyDoc links in ml are in non-standard format.

[GitHub] spark pull request: [SPARK-14684] [SPARK-15026] [SQL] Disallow Dro...

2016-05-04 Thread gatorsmile
Github user gatorsmile commented on the pull request: https://github.com/apache/spark/pull/12801#issuecomment-217070045 @andrewor14 For the defect mentioned in SPARK-14684, you can reproduce it by using the following statement: ``` sql(s"ALTER TABLE $externalTab DROP PARTITION

[GitHub] spark pull request: [SPARK-14897] [SQL] [WIP] upgrade to jetty 9.2...

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

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

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

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

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

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12887#issuecomment-217069905 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-15045] [CORE] Remove dead code in TaskM...

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

[GitHub] spark pull request: [SPARK-15131][SQL] Shutdown StateStore managem...

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

[GitHub] spark pull request: [SPARK-15110][SparkR] Implement repartitionByC...

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

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

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

[GitHub] spark pull request: [SPARK-14997][SQL] Fixed FileCatalog to return...

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/12856#issuecomment-217065010 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-14997][SQL] Fixed FileCatalog to return...

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

[GitHub] spark pull request: [SPARK-14997][SQL] Fixed FileCatalog to return...

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

[GitHub] spark pull request: [SPARK-12837][CORE] reduce network IO for accu...

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

[GitHub] spark pull request: [SPARK-14434][ML]:User guide doc and examples ...

2016-05-04 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/12788#issuecomment-217064788 @sethah It seems that these examples follow `ml.kmeans`'s examples. I agreed that it's better to keep in line with mllib's, especially the output information.

[GitHub] spark pull request: [SPARK-14670][SQL] Allow updating SQLMetrics o...

2016-05-04 Thread cloud-fan
Github user cloud-fan commented on the pull request: https://github.com/apache/spark/pull/12427#issuecomment-217064757 I have the same concern with @zsxwing , what if we execute a DataFrame multiple times? --- If your project is set up for it, you can reply to this email and have

  1   2   3   4   5   6   7   8   9   10   >