Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14705
I think it's ok to put @param ... on top of the function in the order we
want it. Or @param na.rm for sd, var etc. Yes it is a bit odd to have param in
the documentation block that is not in
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/12436
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 does not have this feature
enabled and wishes so, or if the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14666
**[Test build #64083 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64083/consoleFull)**
for PR 14666 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14666
**[Test build #64084 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64084/consoleFull)**
for PR 14666 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14666
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64084/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14721
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 junyangq commented on the issue:
https://github.com/apache/spark/pull/14705
That makes sense. Perhaps this could be done in another 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
Github user davies commented on the issue:
https://github.com/apache/spark/pull/14693
Merging this into master and 2.0 and 1.6 (hopefully), 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
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14721#discussion_r75530744
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -1278,10 +1278,16 @@ class AstBuilder extends
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14650#discussion_r75530775
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -1962,6 +1962,26 @@ private[spark] object Utils extends Logging {
path
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14697
**[Test build #64080 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64080/consoleFull)**
for PR 14697 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14697
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/10896
**[Test build #64079 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64079/consoleFull)**
for PR 10896 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14721
**[Test build #64095 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64095/consoleFull)**
for PR 14721 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14712#discussion_r75513984
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -88,14 +90,30 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14719
**[Test build #64081 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64081/consoleFull)**
for PR 14719 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14712#discussion_r75515983
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -88,14 +90,30 @@ case class
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14705#discussion_r75516177
--- Diff: R/pkg/R/DataFrame.R ---
@@ -932,7 +932,7 @@ setMethod("sample_frac",
#' @param x a SparkDataFrame.
#' @family SparkDataFrame
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523853
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -183,24 +191,18 @@ private[ml] trait DecisionTreeParams extends
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523850
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -183,24 +191,18 @@ private[ml] trait DecisionTreeParams extends
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523858
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -220,32 +222,42 @@ private[ml] object TreeClassifierParams {
final
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523822
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impurity/ApproxBernoulliImpurity.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to the
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523794
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GBTRegressor.scala ---
@@ -38,25 +38,35 @@ import org.apache.spark.sql.{DataFrame, Dataset}
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14155#discussion_r75523912
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -200,22 +375,77 @@ private[spark] class
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523854
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -183,24 +191,18 @@ private[ml] trait DecisionTreeParams extends
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14155#discussion_r75523842
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -200,22 +375,77 @@ private[spark] class
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523771
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/GBTClassifier.scala ---
@@ -42,18 +42,30 @@ import org.apache.spark.sql.types.DoubleType
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523815
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/GradientBoostedTrees.scala
---
@@ -258,11 +258,13 @@ private[spark] object
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523781
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GBTRegressor.scala ---
@@ -17,13 +17,13 @@
package org.apache.spark.ml.regression
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523871
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -501,36 +564,75 @@ private[ml] trait GBTClassifierParams extends
GBTParams
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523805
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/DTStatsAggregator.scala ---
@@ -33,11 +34,13 @@ private[spark] class DTStatsAggregator(
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/14547
One questions I had - this PR creates an inherent coupling between the
impurity used to train the tree and the loss used for boosting. This is not how
I understood tree boost. My impression was
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14038
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64075/
Test PASSed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14155#discussion_r75528185
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -200,22 +375,77 @@ private[spark] class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14038
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/14717
**[Test build #64092 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64092/consoleFull)**
for PR 14717 at commit
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14650
I'm generally fine with this, though one downside is that it introduces a
consistency with other daemon classes such as Master.scala, which only accepts
a properties file. Maybe we should make a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14719
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64082/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14697
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64080/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/14718
**[Test build #64094 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64094/consoleFull)**
for PR 14718 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14719
**[Test build #64082 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64082/consoleFull)**
for PR 14719 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14718
**[Test build #64074 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64074/consoleFull)**
for PR 14718 at commit
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/14204
@nblintao Now that #14382 is merged you can update this, here's a patch of
what you'll need:
`diff --git
a/core/src/main/resources/org/apache/spark/ui/static/executorspage.js
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14666
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64083/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14666
**[Test build #64083 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64083/consoleFull)**
for PR 14666 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14666
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/14649
**[Test build #64086 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64086/consoleFull)**
for PR 14649 at commit
Github user Parth-Brahmbhatt commented on the issue:
https://github.com/apache/spark/pull/14655
@gatorsmile not sure if its the same issue. The issue you are pointing at
talks about storing the actual partition level stats, which could be used by
this PR but until its available we
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14467
**[Test build #64087 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64087/consoleFull)**
for PR 14467 at commit
GitHub user srinathshankar opened a pull request:
https://github.com/apache/spark/pull/14721
[SC-4296][SQL] Change error message for out of range numeric literals
## What changes were proposed in this pull request?
Modifies error message for numeric literals to
Numeric
Github user srinathshankar commented on the issue:
https://github.com/apache/spark/pull/14721
@sameeragarwal @rxin @davies
---
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/14666
**[Test build #64091 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64091/consoleFull)**
for PR 14666 at commit
Github user davies commented on a diff in the pull request:
https://github.com/apache/spark/pull/14693#discussion_r75529799
--- Diff:
core/src/main/java/org/apache/spark/util/collection/unsafe/sort/UnsafeExternalSorter.java
---
@@ -522,7 +522,7 @@ public long spill() throws
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14650#discussion_r75530884
--- Diff:
core/src/main/scala/org/apache/spark/deploy/mesos/MesosClusterDispatcherArguments.scala
---
@@ -102,7 +115,9 @@ private[mesos] class
Github user mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14650#discussion_r75530859
--- Diff:
core/src/main/scala/org/apache/spark/deploy/mesos/MesosClusterDispatcherArguments.scala
---
@@ -102,7 +115,9 @@ private[mesos] class
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/14721#discussion_r75530829
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -1291,28 +1297,32 @@ class AstBuilder extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13650
**[Test build #64093 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64093/consoleFull)**
for PR 13650 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13320
**[Test build #64077 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64077/consoleFull)**
for PR 13320 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14712#discussion_r75515751
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -88,14 +90,30 @@ case class
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14712
So far, the test coverage is weak. Could we add more test cases to cover
all the corner cases? Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jisookim0513 commented on the issue:
https://github.com/apache/spark/pull/10212
@vanzin I updated the 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 feature
enabled
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/14547
@vlad17 Thanks for the PR! I'm not done with a review pass, but I'll go
ahead and send comments from a partial pass.
---
If your project is set up for it, you can reply to this email and have
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/14547
Test gists
* ```setMinInstancesPerNode(10)```: Is this the same value used by gbm by
default?
* Is ```counts.max / counts.sum``` meant to verify that the train/test
splits are identical?
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523882
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -465,33 +497,64 @@ private[ml] trait GBTParams extends
TreeEnsembleParams
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523887
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -465,33 +497,64 @@ private[ml] trait GBTParams extends
TreeEnsembleParams
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14674
I'm fine with this PR, but I'm wondering if it makes more sense to set the
default to one of the values here (probably "TLS"):
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523869
--- Diff: mllib/src/main/scala/org/apache/spark/ml/tree/treeParams.scala ---
@@ -465,33 +497,64 @@ private[ml] trait GBTParams extends
TreeEnsembleParams
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14721
**[Test build #3229 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3229/consoleFull)**
for PR 14721 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14721
**[Test build #64089 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64089/consoleFull)**
for PR 14721 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14384
**[Test build #64090 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64090/consoleFull)**
for PR 14384 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14327
I have also backported this bug fix into 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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14655
How about waiting for a few days until that is delivered? Let us see
whether that might simplify your PR.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64079/
Test PASSed.
---
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/14674
If we document what the default is, I think it's a fair requirement that
the user read the docs and verify that it meets their needs.
---
If your project is set up for it, you can reply to this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14697
**[Test build #64080 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64080/consoleFull)**
for PR 14697 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14712#discussion_r75516336
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -108,4 +126,8 @@ case class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14712#discussion_r75516944
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/AnalyzeTableCommand.scala
---
@@ -88,14 +90,30 @@ case class
Github user BryanCutler closed the pull request at:
https://github.com/apache/spark/pull/13898
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14718
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64074/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14718
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 sethah commented on a diff in the pull request:
https://github.com/apache/spark/pull/14717#discussion_r75519309
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -256,6 +256,15 @@ class LogisticRegression @Since("1.2.0")
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10212
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/10212
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64085/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10212
**[Test build #64085 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64085/consoleFull)**
for PR 10212 at commit
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523820
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impurity/ApproxBernoulliImpurity.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to the
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523836
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impurity/ApproxBernoulliImpurity.scala
---
@@ -0,0 +1,162 @@
+/*
+ * Licensed to the
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523777
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/GBTClassifier.scala ---
@@ -148,11 +154,14 @@ class GBTClassifier @Since("1.4.0") (
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523763
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/GBTClassifier.scala ---
@@ -42,18 +42,30 @@ import org.apache.spark.sql.types.DoubleType
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523809
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/tree/impl/GradientBoostedTrees.scala
---
@@ -258,11 +258,13 @@ private[spark] object
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523798
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GBTRegressor.scala ---
@@ -134,11 +146,15 @@ class GBTRegressor @Since("1.4.0")
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/14547#discussion_r75523785
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GBTRegressor.scala ---
@@ -38,25 +38,35 @@ import org.apache.spark.sql.{DataFrame, Dataset}
Github user nblintao commented on the issue:
https://github.com/apache/spark/pull/14204
@ajbozarth Great! Glad to see your PR merged. Thanks for providing this
patch, I'll fix it days later.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user wangmiao1981 commented on the issue:
https://github.com/apache/spark/pull/14674
@mgummelt I am fine with setting a default value. I have one concern about
it. My motivation is to let the user make sure that they choose a proper
protocol to use which meets their security
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/14718
need to update the test to handle the new levedb
---
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 mgummelt commented on a diff in the pull request:
https://github.com/apache/spark/pull/14650#discussion_r75530239
--- Diff:
core/src/main/scala/org/apache/spark/deploy/mesos/MesosClusterDispatcherArguments.scala
---
@@ -73,6 +82,10 @@ private[mesos] class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14717
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64092/
Test FAILed.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14693
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14717
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/14717
**[Test build #64092 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64092/consoleFull)**
for PR 14717 at commit
1 - 100 of 709 matches
Mail list logo
