Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/12004#discussion_r113967945
--- Diff: docs/cloud-integration.md ---
@@ -0,0 +1,512 @@
+---
+layout: global
+displayTitle: Integration with Cloud Infrastructures
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/12004#discussion_r113969133
--- Diff: docs/cloud-integration.md ---
@@ -0,0 +1,512 @@
+---
+layout: global
+displayTitle: Integration with Cloud Infrastructures
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/12004#discussion_r113970275
--- Diff: docs/cloud-integration.md ---
@@ -0,0 +1,512 @@
+---
+layout: global
+displayTitle: Integration with Cloud Infrastructures
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/12004#discussion_r113976778
--- Diff: docs/cloud-integration.md ---
@@ -0,0 +1,512 @@
+---
+layout: global
+displayTitle: Integration with Cloud Infrastructures
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/12004#discussion_r113976699
--- Diff: docs/cloud-integration.md ---
@@ -0,0 +1,512 @@
+---
+layout: global
+displayTitle: Integration with Cloud Infrastructures
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/12004#discussion_r113976861
--- Diff: docs/cloud-integration.md ---
@@ -0,0 +1,512 @@
+---
+layout: global
+displayTitle: Integration with Cloud Infrastructures
Github user brkyvz commented on the issue:
https://github.com/apache/spark/pull/17804
merging to master and branch-2.1
---
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 user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/17237#discussion_r113983452
--- Diff: python/pyspark/ml/feature.py ---
@@ -1936,6 +1935,14 @@ class StringIndexer(JavaEstimator, HasInputCol,
HasOutputCol, HasHandleInvalid,
Github user zero323 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17783#discussion_r113991432
--- Diff: R/pkg/inst/tests/testthat/test_sparkSQL.R ---
@@ -1478,6 +1481,13 @@ test_that("column functions", {
lapply(
list(list(x = 1,
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/17783
Interesting, I think we got 0.2 from his github
https://github.com/jimhester/lintr
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user zero323 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17783#discussion_r113992412
--- Diff: R/pkg/R/column.R ---
@@ -67,8 +67,7 @@ operators <- list(
"+" = "plus", "-" = "minus", "*" = "multiply", "/" = "divide", "%%" =
"mod",
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17802
You also need to add the new API to MimaExludes.scala
https://github.com/apache/spark/blob/master/project/MimaExcludes.scala#L37-L39
---
If your project is set up for it, you can
Github user anabranch commented on the issue:
https://github.com/apache/spark/pull/17804
yeah it's broken in both, only visible in 1. Not sure if that needs to be
in 2 PRs.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/17804#discussion_r113965033
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -26,13 +26,16 @@ import
Github user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/17804#discussion_r113966339
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -108,6 +111,22 @@ class KafkaSinkSuite
Github user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/17804#discussion_r113966362
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -108,6 +111,22 @@ class KafkaSinkSuite
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17803
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17803
**[Test build #76269 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76269/testReport)**
for PR 17803 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17803
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76269/
Test FAILed.
---
Github user markgrover commented on the issue:
https://github.com/apache/spark/pull/17790
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 not have this feature
enabled and
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17711#discussion_r113972077
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -547,6 +547,10 @@ valueExpression
|
Github user wangmiao1981 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17640#discussion_r113974703
--- Diff: R/pkg/R/serialize.R ---
@@ -83,6 +83,7 @@ writeObject <- function(con, object, writeType = TRUE) {
Date = writeDate(con,
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/12004#discussion_r113976264
--- Diff: docs/cloud-integration.md ---
@@ -0,0 +1,512 @@
+---
+layout: global
+displayTitle: Integration with Cloud Infrastructures
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17802
**[Test build #76277 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76277/testReport)**
for PR 17802 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/17780
Can we at least include the serde?
---
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 user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/17804
@anabranch can you open this against branch-2.1 instead of master?
---
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 user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/17804#discussion_r113966304
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -108,6 +111,22 @@ class KafkaSinkSuite
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17804
**[Test build #76275 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76275/testReport)**
for PR 17804 at commit
Github user brkyvz commented on the issue:
https://github.com/apache/spark/pull/17804
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 and wishes so, or if the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17785
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17785
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76270/
Test PASSed.
---
GitHub user wangmiao1981 opened a pull request:
https://github.com/apache/spark/pull/17805
[SparkR][DOC][SPARK-20477]: Document R bisecting k-means in R programming
guide
## What changes were proposed in this pull request?
Add hyper link in the SparkR programming guide.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17805
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76276/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17805
**[Test build #76276 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76276/testReport)**
for PR 17805 at commit
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/12004#discussion_r113979701
--- Diff: docs/cloud-integration.md ---
@@ -0,0 +1,512 @@
+---
+layout: global
+displayTitle: Integration with Cloud Infrastructures
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/12004#discussion_r113979717
--- Diff: docs/cloud-integration.md ---
@@ -0,0 +1,512 @@
+---
+layout: global
+displayTitle: Integration with Cloud Infrastructures
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15009
**[Test build #76278 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76278/testReport)**
for PR 15009 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17783
**[Test build #76279 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76279/testReport)**
for PR 17783 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17783
**[Test build #76280 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76280/testReport)**
for PR 17783 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15009
**[Test build #76274 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76274/testReport)**
for PR 15009 at commit
Github user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/17804#discussion_r113966069
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -26,13 +26,16 @@ import
Github user anabranch commented on a diff in the pull request:
https://github.com/apache/spark/pull/17804#discussion_r113966089
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -26,13 +26,16 @@ import
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17800#discussion_r113971640
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala
---
@@ -653,7 +653,13 @@ object JdbcUtils extends
Github user wangmiao1981 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17640#discussion_r113972686
--- Diff: R/pkg/R/serialize.R ---
@@ -83,6 +83,7 @@ writeObject <- function(con, object, writeType = TRUE) {
Date = writeDate(con,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17805
**[Test build #76276 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76276/testReport)**
for PR 17805 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17805
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 does not have this feature
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/17790
If there are no more concerns I'll push this in the PM.
---
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 user zsxwing commented on the issue:
https://github.com/apache/spark/pull/17790
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 and wishes so, or if the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15009
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76274/
Test PASSed.
---
Github user zero323 commented on the issue:
https://github.com/apache/spark/pull/17783
OK, let's remove example output for now and leave `lintr` issues for
another day.
I wonder if we could actually remove `\dontrun`, push `sparkR.session` into
`\dontshow` and still keep
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15009
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 does not have this feature
Github user brkyvz commented on the issue:
https://github.com/apache/spark/pull/17804
@hvanhovell We need this fix in master as well. It somehow works magically
in master, but is broken for sure in 2.1
---
If your project is set up for it, you can reply to this email and have your
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17785
**[Test build #76270 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76270/testReport)**
for PR 17785 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17804
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76275/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17804
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17804
**[Test build #76275 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76275/testReport)**
for PR 17804 at commit
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/17804#discussion_r113965269
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -108,6 +111,22 @@ class KafkaSinkSuite extends
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/17804#discussion_r113965053
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -26,13 +26,16 @@ import
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/17804#discussion_r113965237
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -108,6 +111,22 @@ class KafkaSinkSuite extends
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/17804#discussion_r113965109
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaSinkSuite.scala
---
@@ -108,6 +111,22 @@ class KafkaSinkSuite extends
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/12004#discussion_r113971968
--- Diff: docs/cloud-integration.md ---
@@ -0,0 +1,512 @@
+---
+layout: global
+displayTitle: Integration with Cloud Infrastructures
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17804
---
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 user phatak-dev commented on the issue:
https://github.com/apache/spark/pull/17802
Thanks for the feedback. Updated PR according your comments.
---
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 user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17802
**[Test build #76277 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76277/testReport)**
for PR 17802 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17802
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76277/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17802
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17783
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76279/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17783
**[Test build #76279 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76279/testReport)**
for PR 17783 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17783
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 does not have this feature
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/12004#discussion_r113995508
--- Diff: docs/cloud-integration.md ---
@@ -0,0 +1,512 @@
+---
+layout: global
+displayTitle: Integration with Cloud Infrastructures
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17783
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76290/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17783
**[Test build #76290 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76290/testReport)**
for PR 17783 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17783
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17783
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76291/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17783
**[Test build #76291 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76291/testReport)**
for PR 17783 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17783
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 does not have this feature
Github user facaiy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17556#discussion_r114043563
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/RandomForest.scala ---
@@ -1009,10 +1009,24 @@ private[spark] object RandomForest extends
Github user facaiy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17556#discussion_r114043568
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala ---
@@ -138,9 +169,10 @@ class RandomForestSuite extends SparkFunSuite
Github user facaiy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17556#discussion_r114043558
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/RandomForest.scala ---
@@ -1037,7 +1051,10 @@ private[spark] object RandomForest extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15009
**[Test build #76288 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76288/testReport)**
for PR 15009 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15009
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15009
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76288/
Test PASSed.
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17770
@hvanhovell @rxin I've updated this accordingly. Do you have more comments
on this? Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user lw-lin commented on the issue:
https://github.com/apache/spark/pull/17346
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 have this feature
enabled and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17793
**[Test build #76292 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76292/testReport)**
for PR 17793 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17795
@ajbozarth , I checked the UI related to attemptId, seems fine without
issue. Can you please point out in which code potentially has the regression?
Thanks!
---
If your project is set up for
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17795
I assume only event log download will be effected with #17582 .
---
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 user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/15435#discussion_r114043519
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -1231,6 +1295,109 @@ class
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17736
@cloud-fan although @hvanhovell haven't comment yet, I will go to fix the
inconsistency first and see if we have defined tests against it.
---
If your project is set up for it, you can reply to
Github user facaiy commented on the issue:
https://github.com/apache/spark/pull/17556
For a (train) sample of continuous series, say {x0, x1, x2, x3, ..., x100}.
Now spark select quantile as split point.
Suppose 10-quantiles is used, and x2 is 1st quantile, and x10 is 2nd
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17783
**[Test build #76290 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76290/testReport)**
for PR 17783 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17783
**[Test build #76291 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76291/testReport)**
for PR 17783 at commit
Github user facaiy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17556#discussion_r114043439
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/tree/impl/RandomForestSuite.scala ---
@@ -112,9 +138,11 @@ class RandomForestSuite extends SparkFunSuite
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17793
**[Test build #76292 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76292/testReport)**
for PR 17793 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17793
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76292/
Test PASSed.
---
Github user facaiy commented on the issue:
https://github.com/apache/spark/pull/17556
By the way, it's safe to use mean value as it is match the other libraries.
If requested, I'd like to modify the PR.
---
If your project is set up for it, you can reply to this email and have your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17793
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 does not have this feature
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/17723
@mridulm if you're talking about
`ServiceCredentialProvider.obtainCredentials`, which is a public interface that
exposes UGI (`Credentials` in this case), then sure, it's probably worth to
tweak it
Github user danielyli commented on a diff in the pull request:
https://github.com/apache/spark/pull/17793#discussion_r114044512
--- Diff: mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala
---
@@ -910,26 +944,127 @@ object ALS extends DefaultParamsReadable[ALS] with
GitHub user zero323 opened a pull request:
https://github.com/apache/spark/pull/17807
[SPARK-20532][SPARKR] Implement grouping and grouping_id
## What changes were proposed in this pull request?
Adds R wrappers for:
- `o.a.s.sql.functions.grouping` as
1 - 100 of 369 matches
Mail list logo
