Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14608
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/14608
Shouldn't we wrap literals with () to be safe?
---
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/14559
**[Test build #63664 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63664/consoleFull)**
for PR 14559 at commit
[`186a582`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14447
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14447
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63668/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14447
**[Test build #63668 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63668/consoleFull)**
for PR 14447 at commit
[`5f7fd00`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14452
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14452
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63665/
Test FAILed.
---
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/14585
LGTM, merging to master and branch-2.0. 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 f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14452
**[Test build #63665 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63665/consoleFull)**
for PR 14452 at commit
[`9fe1fbe`](https://github.com/apache/spark/commit/
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14585
Did this make it into 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 GraceH commented on a diff in the pull request:
https://github.com/apache/spark/pull/14559#discussion_r74546716
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -423,6 +423,10 @@ final class DataFrameWriter[T] private[sql](ds:
Dataset
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14365
@cloud-fan So do we have decision on this? A simpler approach to disable
the conversion if the schema is inconsistent, or a complex one to work around
this Hive bug?
---
If your project is set up f
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/14559#discussion_r74546013
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -423,6 +423,10 @@ final class DataFrameWriter[T] private[sql](ds:
Da
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14617
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63657/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14617
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14447
**[Test build #63668 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63668/consoleFull)**
for PR 14447 at commit
[`5f7fd00`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14182
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14182
**[Test build #63663 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63663/consoleFull)**
for PR 14182 at commit
[`edd1ce0`](https://github.com/apache/spark/commit/
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14392
Yeah I am not sure `mvnormalmixEM` is very descriptive. @junyangq Any
opinions on the name here ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on Gi
Github user NarineK commented on the issue:
https://github.com/apache/spark/pull/14431
yes, @shivaram , that will be one way to do.
Basically, adding a new public function to `RelationalGroupedDataset` which
will return the column names.
If it is fine from SQL perspective, mayb
Github user GraceH commented on a diff in the pull request:
https://github.com/apache/spark/pull/14559#discussion_r74542628
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCOptions.scala
---
@@ -20,14 +20,21 @@ package org.apache.spark.sql.exec
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14559
**[Test build #63662 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63662/consoleFull)**
for PR 14559 at commit
[`4fb5e55`](https://github.com/apache/spark/commit/
Github user GraceH commented on the issue:
https://github.com/apache/spark/pull/14559
Thanks all. I have added the unit test in JDBCWriterSuite. Any further
comment, please feel free to let me know.
BTW, or we can point the user to check JDBCOptions for further
configuration
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14559
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63662/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14559
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14558
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63654/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14558
**[Test build #63654 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63654/consoleFull)**
for PR 14558 at commit
[`d2c1d64`](https://github.com/apache/spark/commit/
Github user yinxusen commented on the issue:
https://github.com/apache/spark/pull/14229
@felixcheung I add some aliases for spark.lda related functions. However, I
am not quite understand it. From
[here](https://cran.r-project.org/web/packages/roxygen2/vignettes/rd.html) I
can see th
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14571
Hi, @rxin .
I added test files for window functions for SQLQueryTestSuite and removed
the old `WindowQuerySuite.scala`. Could you review this again?
---
If your project is set up for it,
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/14613
@shivaram Sure. I will add unit tests.
---
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/14452
**[Test build #63658 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63658/consoleFull)**
for PR 14452 at commit
[`bdb6e84`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14617
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63656/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14617
**[Test build #63656 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63656/consoleFull)**
for PR 14617 at commit
[`a5e9d46`](https://github.com/apache/spark/commit/
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/14452
@gatorsmile I've made some changes. I will update this soon.
The optimized plan for the query is:
Join Inner
:- Join Inner
: :- CommonSubquery [a#226, b#227, a#24
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14617
**[Test build #63656 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63656/consoleFull)**
for PR 14617 at commit
[`a5e9d46`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14608
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63652/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14608
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14608
**[Test build #63652 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63652/consoleFull)**
for PR 14608 at commit
[`908253b`](https://github.com/apache/spark/commit/
Github user junyangq commented on a diff in the pull request:
https://github.com/apache/spark/pull/14384#discussion_r74536489
--- Diff: R/pkg/R/mllib.R ---
@@ -632,3 +642,159 @@ setMethod("predict", signature(object =
"AFTSurvivalRegressionModel"),
function(object, n
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14102
---
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 yanboliang commented on the issue:
https://github.com/apache/spark/pull/14551
@nicklavers Please also change seed for ```GaussianMixture``` doctest in
```python/pyspark/ml/clustering.py```. And check whether we need to change seed
for ```KMeans``` doctest. Thanks.
---
If
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13146
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14616
**[Test build #63655 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63655/consoleFull)**
for PR 14616 at commit
[`4087365`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13409
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 clockfly opened a pull request:
https://github.com/apache/spark/pull/14616
[SPARK-16955][SQL] Fix analysis error when using ordinal in ORDER BY or
GROUP BY
## What changes were proposed in this pull request?
This PR adds two unresolved expressions to represent t
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14558
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14609
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63650/
Test FAILed.
---
Github user SaintBacchus commented on the issue:
https://github.com/apache/spark/pull/14530
I will re-run this case, and dig into why the executor will double register.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14607
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/14607
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63646/
Test PASSed.
---
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14433
Can we have this like
[SparkSubmitAction](https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala#L55)
that extends `Enumeration`?
---
If your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14586
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 the issue:
https://github.com/apache/spark/pull/14447
so there are a few competing implementation in R and `mlp` might not be a
super relevant name. @shivaram @mengxr any thought on `spark.mlp` here?
---
If your project is set up for it, you can r
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14447#discussion_r74530136
--- Diff: R/pkg/R/mllib.R ---
@@ -533,6 +626,26 @@ setMethod("write.ml", signature(object =
"KMeansModel", path = "character"),
invisibl
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14447#discussion_r74530039
--- Diff: R/pkg/R/mllib.R ---
@@ -414,6 +421,92 @@ setMethod("predict", signature(object = "KMeansModel"),
return(dataFrame(callJMethod(o
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14447#discussion_r74530043
--- Diff: R/pkg/R/mllib.R ---
@@ -414,6 +421,92 @@ setMethod("predict", signature(object = "KMeansModel"),
return(dataFrame(callJMethod(o
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14447#discussion_r74529955
--- Diff: R/pkg/R/mllib.R ---
@@ -487,7 +580,7 @@ setMethod("write.ml", signature(object =
"NaiveBayesModel", path = "character"),
#' @rdname spark.
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14447#discussion_r74529891
--- Diff: R/pkg/R/mllib.R ---
@@ -414,6 +421,92 @@ setMethod("predict", signature(object = "KMeansModel"),
return(dataFrame(callJMethod(o
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14447#discussion_r74529897
--- Diff: R/pkg/R/mllib.R ---
@@ -414,6 +421,92 @@ setMethod("predict", signature(object = "KMeansModel"),
return(dataFrame(callJMethod(o
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14447#discussion_r74529661
--- Diff: R/pkg/R/mllib.R ---
@@ -414,6 +421,92 @@ setMethod("predict", signature(object = "KMeansModel"),
return(dataFrame(callJMethod(o
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14612
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63645/
Test PASSed.
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14522#discussion_r74529416
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2121,7 +2121,7 @@ setMethod("arrange",
})
#' @rdname arrange
-#' @name orderBy
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14447#discussion_r74529446
--- Diff: R/pkg/R/mllib.R ---
@@ -53,6 +53,13 @@ setClass("AFTSurvivalRegressionModel",
representation(jobj = "jobj"))
#' @note KMeansModel since 2.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14612
**[Test build #63645 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63645/consoleFull)**
for PR 14612 at commit
[`71399f1`](https://github.com/apache/spark/commit/
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14384#discussion_r74527784
--- Diff: R/pkg/R/mllib.R ---
@@ -632,3 +642,159 @@ setMethod("predict", signature(object =
"AFTSurvivalRegressionModel"),
function(object
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14182#discussion_r74527541
--- Diff: R/pkg/R/mllib.R ---
@@ -292,6 +299,85 @@ setMethod("summary", signature(object =
"NaiveBayesModel"),
return(list(apriori = apr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14608
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63644/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14608
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/14608
**[Test build #63644 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63644/consoleFull)**
for PR 14608 at commit
[`154abba`](https://github.com/apache/spark/commit/
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14384#discussion_r74527216
--- Diff: R/pkg/R/mllib.R ---
@@ -632,3 +642,159 @@ setMethod("predict", signature(object =
"AFTSurvivalRegressionModel"),
function(object
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14614
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/14614
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63649/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14609
**[Test build #63650 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63650/consoleFull)**
for PR 14609 at commit
[`75b6c22`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14116
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14116
**[Test build #63641 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63641/consoleFull)**
for PR 14116 at commit
[`dc5d1dc`](https://github.com/apache/spark/commit/
Github user yinxusen commented on the issue:
https://github.com/apache/spark/pull/14229
@felixcheung Yes. Sorry I missed the email.
---
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 featur
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14426
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/14426
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63639/
Test PASSed.
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14182#discussion_r74526172
--- Diff: R/pkg/R/mllib.R ---
@@ -292,6 +299,83 @@ setMethod("summary", signature(object =
"NaiveBayesModel"),
return(list(apriori = apr
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74526303
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,109 @@ private[mesos] trait MesosSchedule
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74526278
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,109 @@ private[mesos] trait MesosSchedule
Github user skonto commented on a diff in the pull request:
https://github.com/apache/spark/pull/11157#discussion_r74526246
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosSchedulerUtils.scala
---
@@ -358,6 +376,109 @@ private[mesos] trait MesosSchedule
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/14561
Agreed with @jerryshao. @lshmouse could you submit the whole refactoring
PR in order to show why this one is necessary? It's better to not refactor
stable code paths unless there is a strong reason
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14182#discussion_r74525862
--- Diff: R/pkg/R/mllib.R ---
@@ -292,6 +299,85 @@ setMethod("summary", signature(object =
"NaiveBayesModel"),
return(list(apriori = apr
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14182#discussion_r74525897
--- Diff: R/pkg/R/mllib.R ---
@@ -292,6 +299,85 @@ setMethod("summary", signature(object =
"NaiveBayesModel"),
return(list(apriori = apr
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14613
@wangmiao1981 Thanks for the PR. Could we add a couple of test cases for
this ? It'll also help me understand what is the expected behavior -- one of
them could be for `collect` with decimals and a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14613
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63648/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14613
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 clockfly commented on the issue:
https://github.com/apache/spark/pull/14546
I think a proper fix will be marking ordinal unresolved, the ordinal can
exists in group by or order by expression.
Then we can make sure the ResolveAggregateFunctions and other analyzer
r
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14392
btw, I think it'll be great to get some feedback on the naming of this.
As per SPARK-14831, should we go with a more Spark specific name like
`gaussianmixture` rather than a R one? How well k
Github user clockfly commented on the issue:
https://github.com/apache/spark/pull/14546
I think the root cause is that the Aggregate operator is treated as
resolved if even it has group by ordinals.
For example:
```
'Filter ('a > 0)
+- Aggregate [2], [count(1) A
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14609
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63638/
Test FAILed.
---
GitHub user robert3005 opened a pull request:
https://github.com/apache/spark/pull/14615
make toJSON not go through rdd form but operate on dataset always
## What changes were proposed in this pull request?
Don't convert toRdd when doing toJSON
## How was this patch
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14609
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14571
I see. Then, I'll include only that today.
---
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 fe
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/12930
Do we still need this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wis
Github user clockfly commented on the issue:
https://github.com/apache/spark/pull/14546
@dongjoon-hyun The exception was muted by line:
https://github.com/apache/spark/pull/14546/files#diff-57b3d87be744b7d79a9beacf8e5e5eb2R1257
If you add some log message, you wils find i
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14586
**[Test build #3222 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3222/consoleFull)**
for PR 14586 at commit
[`a785c01`](https://github.com/apache/spark/commit/
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/14611
Closing this one. Just found another issue with the current implementation
and will reopen in the future
---
If your project is set up for it, you can reply to this email and have your
reply appear
1 - 100 of 527 matches
Mail list logo