Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14866
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64868/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14940
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 feat
Github user srinathshankar commented on the issue:
https://github.com/apache/spark/pull/14866
I'll update the python and R APIs in a follow up. Right now in python and R
a cross join is done if no join exprs/columns and join types are specified. It
would be good to require explicit cr
GitHub user bookling opened a pull request:
https://github.com/apache/spark/pull/14940
[SPARK-17383][GRAPHX]LabelPropagation
In the labelPropagation of graphx lib, node is initialized with a unique
label and at every step each node adopts the label that most of its
neighbors cur
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/14931
What if we added a flag to SlaveLost indicating if we think the entire host
is lost? In many cases that should be true, if the event originated from worker
loss or Mesos slave loss events.
---
If yo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14938
**[Test build #64867 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64867/consoleFull)**
for PR 14938 at commit
[`b57bbb6`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14931
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64862/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14931
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14931
**[Test build #64862 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64862/consoleFull)**
for PR 14931 at commit
[`2430b69`](https://github.com/apache/spark/commit/
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/14797#discussion_r77394975
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -479,13 +480,23 @@ case class DataSource(
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14939
Ah thanks - I didn't notice this while merging the earlier PR.
---
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 n
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14854
**[Test build #64866 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64866/consoleFull)**
for PR 14854 at commit
[`32c3959`](https://github.com/apache/spark/commit/3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14881
**[Test build #64865 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64865/consoleFull)**
for PR 14881 at commit
[`caeb91e`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14828
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14828
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64861/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14828
**[Test build #64861 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64861/consoleFull)**
for PR 14828 at commit
[`302f55f`](https://github.com/apache/spark/commit/
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14939
---
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 ena
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14939
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
e
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14881#discussion_r77388738
--- Diff: R/pkg/R/mllib.R ---
@@ -1308,3 +1315,104 @@ setMethod("write.ml", signature(object =
"ALSModel", path = "character"),
function(o
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14939
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64864/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14939
**[Test build #64864 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64864/consoleFull)**
for PR 14939 at commit
[`6a19a5d`](https://github.com/apache/spark/commit/
Github user robert3005 commented on the issue:
https://github.com/apache/spark/pull/14573
Agree it would be subsumed and it looks pretty cool. I didn't know you can
make it asynchronous also you want to avoid spinning too many tasks since these
consume resources and block other jobs.
Github user atronchi commented on the issue:
https://github.com/apache/spark/pull/10970
How can we workaround this problem when using 1.6.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 th
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14718
---
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 ena
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14718
I thought you were going to merge it yourself, but since you didn't...
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
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/14718
No luck merging to 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 have this feature
enabled and wishes s
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/14573
Nice change, but I think that at least some of the benefit of this will be
subsumed by #14854, my patch which allows `take()` to cancel the running job as
soon as enough output is produced.
---
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14866
LGTM - pending jenkins.
---
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 wish
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14907#discussion_r77384493
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala
---
@@ -154,6 +163,297 @@ object JdbcUtils extends Loggi
Github user junyangq commented on a diff in the pull request:
https://github.com/apache/spark/pull/14881#discussion_r77384344
--- Diff: R/pkg/R/mllib.R ---
@@ -1308,3 +1315,104 @@ setMethod("write.ml", signature(object =
"ALSModel", path = "character"),
function(obje
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14907#discussion_r77384218
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRDD.scala
---
@@ -546,33 +281,33 @@ private[jdbc] class JDBCRDD(
Github user zhaoyunjiong commented on a diff in the pull request:
https://github.com/apache/spark/pull/14887#discussion_r77383971
--- Diff:
common/network-yarn/src/main/java/org/apache/spark/network/yarn/YarnShuffleService.java
---
@@ -25,6 +25,8 @@
import com.google.common.c
Github user srinathshankar commented on a diff in the pull request:
https://github.com/apache/spark/pull/14912#discussion_r77383932
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/FilterPushdownSuite.scala
---
@@ -171,6 +172,27 @@ class FilterPushdown
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14866#discussion_r77383666
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2265,7 +2265,7 @@ setMethod("join",
signature(x = "SparkDataFrame", y = "SparkDataFrame"),
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14939
**[Test build #64864 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64864/consoleFull)**
for PR 14939 at commit
[`6a19a5d`](https://github.com/apache/spark/commit/6
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14934
---
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 ena
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/14939
[SPARK-17376][SPARKR] followup - change since version
## What changes were proposed in this pull request?
change since version in doc
## How was this patch tested?
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14938
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14938
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64859/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14938
**[Test build #64859 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64859/consoleFull)**
for PR 14938 at commit
[`7ab9de0`](https://github.com/apache/spark/commit/
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14934
LGTM. Merging into 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 does not have this feature
enab
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14934#discussion_r77381083
--- Diff: R/pkg/R/functions.R ---
@@ -2876,7 +2876,8 @@ setMethod("randn", signature(seed = "numeric"),
#' regexp_extract
#'
-#' Extract
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14934#discussion_r77380815
--- Diff: R/pkg/R/functions.R ---
@@ -2876,7 +2876,8 @@ setMethod("randn", signature(seed = "numeric"),
#' regexp_extract
#'
-#' Extra
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14935
---
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 ena
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14935
Merged into 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 does not have this feature
enabled and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14866
**[Test build #64863 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64863/consoleFull)**
for PR 14866 at commit
[`7f3d67f`](https://github.com/apache/spark/commit/7
Github user srinathshankar commented on a diff in the pull request:
https://github.com/apache/spark/pull/14866#discussion_r77380304
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2265,7 +2265,7 @@ setMethod("join",
signature(x = "SparkDataFrame", y = "SparkDataFrame"),
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14857
---
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 ena
Github user davies commented on the issue:
https://github.com/apache/spark/pull/14857
Merging into master and 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 have this feature
enabled and wish
Github user davies commented on the issue:
https://github.com/apache/spark/pull/14857
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 featur
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14931
**[Test build #64862 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64862/consoleFull)**
for PR 14931 at commit
[`2430b69`](https://github.com/apache/spark/commit/2
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/14931#discussion_r77378981
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.scala ---
@@ -346,15 +346,16 @@ private[spark] class TaskSchedulerImpl(
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14867
cc @srinathshankar
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14907
---
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 ena
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14712
Below is a test case for a table with empty column. Could you also add it
here?
```Scala
test("statistics collection of a table with zero column") {
val table_no_cols = "ta
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14712
LGTM except two minor comments about test cases.
---
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14907
LGTM. Merging to master. Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14907#discussion_r77377425
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala
---
@@ -154,6 +163,297 @@ object JdbcUtils extends Logg
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14907#discussion_r77377079
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala
---
@@ -154,6 +163,288 @@ object JdbcUtils extends Logg
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14907#discussion_r77375713
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRDD.scala
---
@@ -546,33 +281,33 @@ private[jdbc] class JDBCRDD(
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14927
@davies are we making this assumption anywhere else in our unsafe code?
---
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 pro
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14934#discussion_r77375452
--- Diff: R/pkg/R/functions.R ---
@@ -2876,7 +2876,8 @@ setMethod("randn", signature(seed = "numeric"),
#' regexp_extract
#'
-#' Extract
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14433
LGTM. Thanks @wangmiao1981 -- I'll keep this open for a bit to see if there
are any more comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14935
LGTM. I actually think this might also be useful in 2.0.1 ?
cc @junyangq
---
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 user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14712#discussion_r77373066
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/StatisticsSuite.scala ---
@@ -168,6 +170,137 @@ class StatisticsSuite extends QueryTest with
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14924
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64856/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14924
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
e
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/14842#discussion_r77372825
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeInMemorySorter.java
---
@@ -333,11 +333,12 @@ public UnsafeSorterIterator ge
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14924
**[Test build #64856 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64856/consoleFull)**
for PR 14924 at commit
[`7583070`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14923
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14923
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64858/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14923
**[Test build #64858 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64858/consoleFull)**
for PR 14923 at commit
[`cea5cdf`](https://github.com/apache/spark/commit/
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14921
LGTM again : )
---
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 vanzin commented on the issue:
https://github.com/apache/spark/pull/14931
Hmm... as far as I understand, `SlaveLost` does not mean "remote machine
has died", it means "remote executor has died", which does not mean that the
shuffle files were lost. Unless you actually try
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14828
**[Test build #64861 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64861/consoleFull)**
for PR 14828 at commit
[`302f55f`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14828
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14828
**[Test build #64860 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64860/consoleFull)**
for PR 14828 at commit
[`59b359c`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14828
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64860/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14828
**[Test build #64860 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64860/consoleFull)**
for PR 14828 at commit
[`59b359c`](https://github.com/apache/spark/commit/5
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/14938
Lgtm. I wonder if it's also useful to set the truncation threshold lower in
test? Though maybe that's a nuisance
---
If your project is set up for it, you can reply to this email and have your
reply
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14573
---
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 ena
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14933
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64855/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14933
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
e
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/14573
LGTM - merging to master. Thanks for working on 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14933
**[Test build #64855 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64855/consoleFull)**
for PR 14933 at commit
[`de488ce`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14938
**[Test build #64859 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64859/consoleFull)**
for PR 14938 at commit
[`7ab9de0`](https://github.com/apache/spark/commit/7
GitHub user hvanhovell opened a pull request:
https://github.com/apache/spark/pull/14938
Fix ArrayType and MapType CatalogString.
## What changes were proposed in this pull request?
the `catalogString` for `ArrayType` and `MapType` currently calls the
`simpleString` method on it
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/14922#discussion_r77362198
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/param/shared/sharedParams.scala ---
@@ -405,5 +405,9 @@ private[ml] trait HasAggregationDepth extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14923
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14923
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64857/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14923
**[Test build #64857 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64857/consoleFull)**
for PR 14923 at commit
[`be59825`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14923
**[Test build #64858 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64858/consoleFull)**
for PR 14923 at commit
[`cea5cdf`](https://github.com/apache/spark/commit/c
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/14923#discussion_r77361651
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/stat/MultivariateOnlineSummarizer.scala
---
@@ -231,9 +231,9 @@ class MultivariateOnlineSummarize
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/14922#discussion_r77358835
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -295,6 +295,13 @@ class LogisticRegression @Since("1.
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14521
thanks, merging to master and 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 have this feature
enable
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14923#discussion_r77357355
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/stat/MultivariateOnlineSummarizer.scala
---
@@ -231,9 +231,9 @@ class MultivariateOnlineSummarizer exte
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14922#discussion_r77357216
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/param/shared/sharedParams.scala ---
@@ -405,5 +405,9 @@ private[ml] trait HasAggregationDepth extends Param
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14922#discussion_r77357109
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -295,6 +295,13 @@ class LogisticRegression @Since("1.2.0")
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14521
---
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 ena
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/14922#discussion_r77356253
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -295,6 +295,13 @@ class LogisticRegression @Since("1.
201 - 300 of 467 matches
Mail list logo
