Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13758
**[Test build #61361 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61361/consoleFull)**
for PR 13758 at commit
Github user ScrapCodes commented on the issue:
https://github.com/apache/spark/pull/13839
retest this please.
Failed tests: (may be flaky.)
```
- missing checkpoint block fails with informative message *** FAILED ***
(41 milliseconds)
[info] Collect should have
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61362/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13680
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/13680
**[Test build #61362 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61362/consoleFull)**
for PR 13680 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/13858#discussion_r68716660
--- Diff: core/src/main/scala/org/apache/spark/scheduler/TaskLocation.scala
---
@@ -64,18 +64,21 @@ private[spark] object TaskLocation {
/**
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/13858#discussion_r68716518
--- Diff: core/src/main/scala/org/apache/spark/scheduler/TaskLocation.scala
---
@@ -64,18 +64,21 @@ private[spark] object TaskLocation {
/**
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13806
**[Test build #61368 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61368/consoleFull)**
for PR 13806 at commit
Github user tmagrino commented on the issue:
https://github.com/apache/spark/pull/13858
I opted to force push a cleaned up version, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/13858
You can always revert the commits in question and push, which become new
commits, but that's fine. They're all squashed in the end. You can also
manually squash and force-push if you like.
---
If
Github user yinxusen commented on a diff in the pull request:
https://github.com/apache/spark/pull/9#discussion_r68715841
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/param/shared/sharedGeneralTypeParams.scala
---
@@ -0,0 +1,34 @@
+/*
--- End diff --
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/13806
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 wishes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13806
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/13806
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61367/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13806
**[Test build #61367 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61367/consoleFull)**
for PR 13806 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13806
**[Test build #61367 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61367/consoleFull)**
for PR 13806 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13941
**[Test build #61366 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61366/consoleFull)**
for PR 13941 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13939
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61365/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13939
**[Test build #61365 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61365/consoleFull)**
for PR 13939 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13939
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 hhbyyh opened a pull request:
https://github.com/apache/spark/pull/13941
[SPARK-16249][ML] Change visibility of Object ml.clustering.LDA to public
for loading
## What changes were proposed in this pull request?
jira: https://issues.apache.org/jira/browse/SPARK-16249
Github user tmagrino commented on the issue:
https://github.com/apache/spark/pull/13858
Ugh, I just saw that I accidentally merged in some other PRs... I'm a bit
new to this, apologies. Any idea how I can amend this?
---
If your project is set up for it, you can reply to this email
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/13806
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 wishes
Github user tmagrino commented on the issue:
https://github.com/apache/spark/pull/13858
My apologies @srowen, I missed the comment somehow! I refactored a little
bit to make it more obvious what's going on there (using `stripPrefix` as you
suggested).
---
If your project is set up
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13806
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/13806
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61357/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13806
**[Test build #61357 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61357/consoleFull)**
for PR 13806 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13939
**[Test build #61365 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61365/consoleFull)**
for PR 13939 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11863
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61359/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11863
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/11863
**[Test build #61359 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61359/consoleFull)**
for PR 11863 at commit
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/13816#discussion_r68712530
--- Diff:
examples/src/main/scala/org/apache/spark/examples/sql/streaming/NetworkEventTimeWindow.scala
---
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/13816#discussion_r68712408
--- Diff:
examples/src/main/scala/org/apache/spark/examples/sql/streaming/NetworkEventTimeWindow.scala
---
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/13816#discussion_r68712352
--- Diff:
examples/src/main/java/org/apache/spark/examples/sql/streaming/JavaStructuredNetworkWordCount.java
---
@@ -0,0 +1,78 @@
+/*
+ * Licensed to
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/13816#discussion_r68712291
--- Diff:
examples/src/main/scala/org/apache/spark/examples/sql/streaming/NetworkEventTimeWindow.scala
---
@@ -0,0 +1,84 @@
+/*
+ * Licensed to the
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/13891
It may not matter much but your test code is a little different than in the
patch, like for copyToTri().
It's optional, but a few comments explaining what addStack does might help
readers.
---
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/13891#discussion_r68711572
--- Diff: mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala
---
@@ -635,6 +636,15 @@ object ALS extends DefaultParamsReadable[ALS] with
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/13891#discussion_r68711415
--- Diff: mllib/src/main/scala/org/apache/spark/ml/recommendation/ALS.scala
---
@@ -1296,6 +1316,9 @@ object ALS extends DefaultParamsReadable[ALS] with
Github user zlpmichelle commented on the issue:
https://github.com/apache/spark/pull/13940
Thanks @hhbyyh @AmplabJenkins!
---
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 yinxusen commented on the issue:
https://github.com/apache/spark/pull/13921
@keypointt You need to add title for `predict` and `write.ml`. Like the
first line below.
```Rd
#' This is title for write.ml
#' @rdname write.ml
#' @export
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13839
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/13839
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61354/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13839
**[Test build #61354 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61354/consoleFull)**
for PR 13839 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13378
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61364/
Test PASSed.
---
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/13921#discussion_r68707718
--- Diff: R/pkg/R/mllib.R ---
@@ -477,24 +480,15 @@ setMethod("write.ml", signature(object =
"GeneralizedLinearRegressionModel", pat
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13378
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/13378
**[Test build #61364 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61364/consoleFull)**
for PR 13378 at commit
Github user keypointt commented on a diff in the pull request:
https://github.com/apache/spark/pull/13921#discussion_r68707667
--- Diff: R/pkg/R/mllib.R ---
@@ -358,19 +369,11 @@ setMethod("summary", signature(object =
"KMeansModel"),
cluster = cluster,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13939
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61358/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13939
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/13939
**[Test build #61358 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61358/consoleFull)**
for PR 13939 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13937
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/13937
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61350/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13937
**[Test build #61350 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61350/consoleFull)**
for PR 13937 at commit
Github user keypointt commented on the issue:
https://github.com/apache/spark/pull/13921
hi @mengxr I tried to modifiy in generics.R
```
#' @rdname spark.kmeans
#' @export
setGeneric("spark.kmeans", function(data, formula, ...) {
standardGeneric("spark.kmeans") })
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13378
**[Test build #61364 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61364/consoleFull)**
for PR 13378 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13906
**[Test build #61363 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61363/consoleFull)**
for PR 13906 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13931
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/13931
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61360/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13931
**[Test build #61360 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61360/consoleFull)**
for PR 13931 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13680
**[Test build #61362 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61362/consoleFull)**
for PR 13680 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13758
**[Test build #61361 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61361/consoleFull)**
for PR 13758 at commit
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/13940
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 dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13906#discussion_r68703430
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/CollapseEmptyPlanSuite.scala
---
@@ -0,0 +1,133 @@
+/*
+ *
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/13858
I had an outstanding comment from the previous PR too:
https://github.com/apache/spark/pull/13857#discussion_r68134544
---
If your project is set up for it, you can reply to this email and have
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13930#discussion_r68702737
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveSessionCatalog.scala ---
@@ -174,6 +175,18 @@ private[sql] class HiveSessionCatalog(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13940
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
GitHub user zlpmichelle opened a pull request:
https://github.com/apache/spark/pull/13940
[SPARK-16241] [ML] model loading backward compatibility for ml NaiveBayes
#16241
## What changes were proposed in this pull request?
model loading backward compatibility for ml
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13903
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/13903
Merging in 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 does not have this feature
enabled and wishes so, or
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/13930#discussion_r68702096
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveSessionCatalog.scala ---
@@ -174,6 +175,18 @@ private[sql] class HiveSessionCatalog(
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/13906
Anyway, thank you for review again, @rxin !
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13938
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/13938
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61355/
Test PASSed.
---
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13906#discussion_r68701978
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1053,6 +1055,41 @@ object PruneFilters extends
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13906#discussion_r68701768
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1053,6 +1055,41 @@ object PruneFilters extends
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/13906#discussion_r68701793
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/CollapseEmptyPlanSuite.scala
---
@@ -0,0 +1,133 @@
+/*
+ *
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13931
**[Test build #61360 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61360/consoleFull)**
for PR 13931 at commit
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/13906#discussion_r68701620
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/CollapseEmptyPlanSuite.scala
---
@@ -0,0 +1,133 @@
+/*
+ * Licensed to
701 - 779 of 779 matches
Mail list logo