Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18157
**[Test build #77577 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77577/testReport)**
for PR 18157 at commit
[`50fda83`](https://github.com/apache/spark/commit/50
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18151
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77574/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18151
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/18151
**[Test build #77574 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77574/testReport)**
for PR 18151 at commit
[`98ffd16`](https://github.com/apache/spark/commit/9
GitHub user wangyum opened a pull request:
https://github.com/apache/spark/pull/18157
[MINOR][SQL] Fix a few function description error.
## What changes were proposed in this pull request?
Fix a few function description error.
## How was this patch tested?
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18156
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 10110346 opened a pull request:
https://github.com/apache/spark/pull/18156
[SPARK-20933][SQL]when the input parameter is float typeï¼the âround â
or âbroundâ function can't work well
## What changes were proposed in this pull request?
spark-sql>select round
Github user liu-zhaokun commented on the issue:
https://github.com/apache/spark/pull/18108
@srowen
I have tested it in my cluster with modify the version of curator,spark
works normally,and the problem I encountered was resolved.
---
If your project is set up for it, you can re
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18151
**[Test build #3769 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3769/testReport)**
for PR 18151 at commit
[`98ffd16`](https://github.com/apache/spark/commit/9
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/18015
@srowen
I think the style exists to provide links that jump to Running Queries or
Completed Queries because it may be well below the fold if there are many
workers.Job page, stage page,
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18015
I don't think those links are worth the vertical space they chew up
---
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 doe
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18108
Generally you need to evaluate the impact of updating a minor version of a
dependency. Does it cause any compatibility problems? What version does Hadoop
use? Does it add new transitive deps? You did
Github user ScrapCodes commented on a diff in the pull request:
https://github.com/apache/spark/pull/18143#discussion_r119277567
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/CachedKafkaConsumer.scala
---
@@ -310,62 +308,45 @@ private[kafka010] obj
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/18129
CC @vanzin to take a 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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18153
**[Test build #77576 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77576/testReport)**
for PR 18153 at commit
[`1b70f29`](https://github.com/apache/spark/commit/1b
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/18129#discussion_r119277116
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/ClientSuite.scala
---
@@ -116,15 +116,16 @@ class ClientSuite extends SparkF
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17617#discussion_r119276245
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala
---
@@ -143,14 +144,18 @@ class SparkHadoopUtil extends Logging {
* Re
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17617#discussion_r119275824
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala
---
@@ -143,14 +144,18 @@ class SparkHadoopUtil extends Logging {
* Re
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17113
**[Test build #77575 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77575/testReport)**
for PR 17113 at commit
[`9a14105`](https://github.com/apache/spark/commit/9a
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r119275552
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -1077,7 +1080,8 @@ class AstBuilder(conf: SQLConf) extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17617#discussion_r119275510
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala
---
@@ -143,14 +144,18 @@ class SparkHadoopUtil extends Logging {
* Re
Github user sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/18151#discussion_r119275319
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LinearSVCSuite.scala ---
@@ -127,6 +127,14 @@ class LinearSVCSuite extends SparkFunSuite wi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18151
**[Test build #77574 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77574/testReport)**
for PR 18151 at commit
[`98ffd16`](https://github.com/apache/spark/commit/98
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17617#discussion_r119275374
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala
---
@@ -143,14 +144,18 @@ class SparkHadoopUtil extends Logging {
* Re
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17603
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/17603
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77569/
Test PASSed.
---
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/18151#discussion_r119275003
--- Diff: R/pkg/R/mllib_classification.R ---
@@ -62,7 +62,7 @@ setClass("NaiveBayesModel", representation(jobj = "jobj"))
#'of
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17603
**[Test build #77569 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77569/testReport)**
for PR 17603 at commit
[`831de22`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18153
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77571/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18153
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/18153
**[Test build #77571 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77571/testReport)**
for PR 18153 at commit
[`209dbaf`](https://github.com/apache/spark/commit/2
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17617#discussion_r119274153
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala
---
@@ -143,14 +144,18 @@ class SparkHadoopUtil extends Logging {
* Re
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18104
---
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 jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/17617
LGTM, cc @cloud-fan @ueshin
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17617
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/17617
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77567/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17617
**[Test build #77567 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77567/testReport)**
for PR 17617 at commit
[`8b16017`](https://github.com/apache/spark/commit/8
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/18147
LGTM except for @cloud-fan's comment.
---
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
enabl
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18154
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/18154
**[Test build #77572 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77572/testReport)**
for PR 18154 at commit
[`f7c5442`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18154
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77572/
Test PASSed.
---
Github user mpjlu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18118#discussion_r119271935
--- Diff: project/MimaExcludes.scala ---
@@ -37,11 +37,15 @@ object MimaExcludes {
// Exclude rules for 2.3.x
lazy val v23excludes = v22excludes
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/18104
LGTM. Thanks @felixcheung for the update and @marmbrus for the ping
---
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 d
Github user mpjlu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18118#discussion_r119271662
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -441,12 +415,44 @@ private[ml] trait RandomForestParams extends
TreeEnsemblePa
Github user mpjlu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18118#discussion_r119271607
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -420,18 +394,18 @@ private[ml] trait RandomForestParams extends
TreeEnsemblePa
Github user mpjlu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18118#discussion_r119271552
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -305,7 +305,7 @@ private[ml] object TreeRegressorParams {
}
pri
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/18153#discussion_r119271567
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
---
@@ -97,20 +97,30 @@ case class UnaryPositive(child: Ex
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18086#discussion_r119271262
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -533,13 +533,16 @@ class AstBuilder(conf: SQLConf) extends
Github user mpjlu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18118#discussion_r119271063
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/DecisionTreeRegressor.scala
---
@@ -99,6 +99,8 @@ class DecisionTreeRegressor @Since("1.4.0")
Github user mpjlu commented on a diff in the pull request:
https://github.com/apache/spark/pull/18118#discussion_r119270999
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/GBTClassifier.scala ---
@@ -136,12 +136,20 @@ class GBTClassifier @Since("1.4.0") (
@
Github user pralabhkumar commented on the issue:
https://github.com/apache/spark/pull/18118
can any one of admin ,please review the pull request. It would be really
helpful. Thanks
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18155
**[Test build #77573 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77573/testReport)**
for PR 18155 at commit
[`5ab4a96`](https://github.com/apache/spark/commit/5a
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/18155
ok to test
---
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 kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r119269498
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -569,7 +569,9 @@ primaryExpression
| '(' namedExpre
Github user kevinyu98 commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r119269209
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -1077,7 +1080,8 @@ class AstBuilder(conf: SQLConf) ext
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/18153#discussion_r119269154
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
---
@@ -97,20 +97,30 @@ case class UnaryPositive(child: Expr
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18152
**[Test build #77570 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77570/testReport)**
for PR 18152 at commit
[`3736992`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18152
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/18152
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77570/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18086#discussion_r119268299
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameHintSuite.scala ---
@@ -0,0 +1,63 @@
+/*
+ * Licensed to the Apache Software Found
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18155
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17770
reverted, thanks for your understanding!
---
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 10110346 opened a pull request:
https://github.com/apache/spark/pull/18155
[SPARK-20876][SQL][Backport-2.2]If the input parameter is float type for
ceil or floor,the result is not we expected
## What changes were proposed in this pull request?
This PR is to back
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18154
**[Test build #77572 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77572/testReport)**
for PR 18154 at commit
[`f7c5442`](https://github.com/apache/spark/commit/f7
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18147#discussion_r119267756
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -191,6 +191,23 @@ def createGlobalTempView(self, name):
"""
self._jdf.createGlob
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18086#discussion_r119267579
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -407,7 +407,7 @@ class AstBuilder(conf: SQLConf) exten
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18086#discussion_r119267472
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameHintSuite.scala ---
@@ -0,0 +1,63 @@
+/*
+ * Licensed to the Apache Software Found
GitHub user zhengruifeng opened a pull request:
https://github.com/apache/spark/pull/18154
[SPARK-20932][ML]CountVectorizer support handle persistence
## What changes were proposed in this pull request?
unpersist RDDs `input` & `wordCounts` after computation
## How was t
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18086#discussion_r119266987
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/parser/PlanParserSuite.scala
---
@@ -525,47 +525,117 @@ class PlanParserSuite extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18086#discussion_r119266645
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/DSLHintSuite.scala
---
@@ -0,0 +1,59 @@
+/*
+ * Licensed to the Apac
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18086#discussion_r119266598
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/DSLHintSuite.scala
---
@@ -0,0 +1,59 @@
+/*
+ * Licensed to the Apac
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18086#discussion_r119266605
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/DSLHintSuite.scala
---
@@ -0,0 +1,59 @@
+/*
+ * Licensed to the Apac
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18147
cc @cloud-fan @ueshin
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18153
**[Test build #77571 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77571/testReport)**
for PR 18153 at commit
[`209dbaf`](https://github.com/apache/spark/commit/20
GitHub user wangyum opened a pull request:
https://github.com/apache/spark/pull/18153
[SPARK-20931][SQL] ABS function support string type.
## What changes were proposed in this pull request?
ABS function support string type. Hive/MySQL support this feature.
Ref:
ht
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r119265202
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -1077,7 +1080,8 @@ class AstBuilder(conf: SQLConf) extends
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r119265297
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -569,7 +569,9 @@ primaryExpression
| '(' namedExpressio
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/17936
Hi @rxin, would you mind take a look?
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17995
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/17995
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77568/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17995
**[Test build #77568 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77568/testReport)**
for PR 17995 at commit
[`0db3a52`](https://github.com/apache/spark/commit/0
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17770
@cloud-fan Ok. No problem for me. 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
en
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17770
Hi @viirya , as this PR already missed the Spark 2.2 release, I'd like to
revert it and re-merge it at the end of Spark 2.3, so that future analyzer
related PRs won't get conflicted when backporti
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17588
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17588
LGTM, merging to master/2.2!
---
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 felixcheung commented on the issue:
https://github.com/apache/spark/pull/18104
ok passes. @shivaram what do you think about 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 t
Github user facaiy commented on a diff in the pull request:
https://github.com/apache/spark/pull/18139#discussion_r119263608
--- Diff: python/pyspark/sql/types.py ---
@@ -187,8 +187,11 @@ def needConversion(self):
def toInternal(self, dt):
if dt is not No
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18152
**[Test build #77570 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77570/testReport)**
for PR 18152 at commit
[`3736992`](https://github.com/apache/spark/commit/37
Github user viirya closed the pull request at:
https://github.com/apache/spark/pull/17998
---
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 viirya commented on the issue:
https://github.com/apache/spark/pull/17998
#18064 is merged. Since it changes relative classes and code paths a lot.
And seems an alternative approach for showing the metrics of writing data out
is better. I'd close this and create a new PR f
GitHub user zhengruifeng opened a pull request:
https://github.com/apache/spark/pull/18152
[SPARK-20930][ML] Destroy broadcasted centers after computing cost in
KMeans
## What changes were proposed in this pull request?
Destroy broadcasted centers after computing cost
## H
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18064
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/17540
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18064
thanks for the review, merging to 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
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/18041
@srowen , this issue existed when reading from metrics.properties conf
file, I think we should fix this part. As for SparkConf part, I don't think it
is necessary to fix.
---
If your project is
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r119261589
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -2616,4 +2617,66 @@ class SQLQuerySuite extends QueryTest with
SharedSQLCon
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/12646#discussion_r119261484
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -461,68 +462,269 @@ case class FindInSet(left:
Github user facaiy commented on the issue:
https://github.com/apache/spark/pull/18120
Thanks, @BryanCutler.
It seems that #17849 copys `Params` from `Estimator` to `Model`
automatically, which is pretty useful. However, `getter` method is still
missing and need to be added manuall
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17603
**[Test build #77569 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77569/testReport)**
for PR 17603 at commit
[`831de22`](https://github.com/apache/spark/commit/83
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/17603
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
1 - 100 of 495 matches
Mail list logo
