Github user zhaorongsheng commented on the issue:
https://github.com/apache/spark/pull/16389
@zsxwing I think it may cause some other problem.
For example, if we got some ExecutorLostFailure and the speculated task was
running on it, the `numRunningTasks` will never be zero.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16422
After rethinking about it, `DESC EXTENDED/FORMATTED COLUMN` discloses the
data patterns/statistics info. These info are pretty sensitive. Not all the
users should be allowed to access it.
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16422
To get the column names and types, we do not need `DESC COLUMN`.
For retrieving the statistics, each vendor has different ways. Normally,
users can access the statistics from the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16341
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/16341
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70762/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16341
**[Test build #70762 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70762/testReport)**
for PR 16341 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16422#discussion_r94267919
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -300,10 +300,21 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16417
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70761/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16417
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/16417
**[Test build #70761 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70761/testReport)**
for PR 16417 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16341
**[Test build #70762 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70762/testReport)**
for PR 16341 at commit
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16441
Thanks for the PR; I do want to get this fixed. However, I don't think
this is the right way to make predictions of probabilities for GBTs. I believe
it should depend on the loss used. E.g.,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16417
**[Test build #70761 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70761/testReport)**
for PR 16417 at commit
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16387
cc @rxin @zsxwing too
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16417
retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16403#discussion_r94264691
--- Diff: common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java
---
@@ -244,6 +251,18 @@ public static void throwException(Throwable t) {
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16405#discussion_r94264468
--- Diff:
examples/src/main/python/mllib/decision_tree_regression_example.py ---
@@ -44,7 +44,7 @@
# Evaluate model on test instances and compute
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16405#discussion_r94263914
--- Diff: dev/lint-python ---
@@ -19,10 +19,8 @@
SCRIPT_DIR="$( cd "$( dirname "$0" )" && pwd )"
SPARK_ROOT_DIR="$(dirname
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16405#discussion_r94263510
--- Diff:
examples/src/main/python/mllib/decision_tree_regression_example.py ---
@@ -44,7 +44,7 @@
# Evaluate model on test instances and
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16424
---
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 liancheng commented on the issue:
https://github.com/apache/spark/pull/16424
OK, I'm merging this to master and branch-2.1. Thanks for the 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16441
**[Test build #70760 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70760/testReport)**
for PR 16441 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16441
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70760/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16441
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/16441
**[Test build #70760 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70760/testReport)**
for PR 16441 at commit
Github user imatiach-msft commented on the issue:
https://github.com/apache/spark/pull/16441
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
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16405#discussion_r94259691
--- Diff:
examples/src/main/python/mllib/decision_tree_regression_example.py ---
@@ -44,7 +44,7 @@
# Evaluate model on test instances and compute
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16405#discussion_r94259548
--- Diff: dev/lint-python ---
@@ -19,10 +19,8 @@
SCRIPT_DIR="$( cd "$( dirname "$0" )" && pwd )"
SPARK_ROOT_DIR="$(dirname "$SCRIPT_DIR")"
Github user michaelkamprath commented on a diff in the pull request:
https://github.com/apache/spark/pull/16403#discussion_r94259189
--- Diff: common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java
---
@@ -22,10 +22,14 @@
import java.lang.reflect.Method;
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16441
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70759/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16441
**[Test build #70759 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70759/testReport)**
for PR 16441 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16441
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 michaelkamprath commented on a diff in the pull request:
https://github.com/apache/spark/pull/16403#discussion_r94257179
--- Diff: common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java
---
@@ -244,6 +251,18 @@ public static void throwException(Throwable t)
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13077
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70758/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16441
**[Test build #70759 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70759/testReport)**
for PR 16441 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13077
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/13077
**[Test build #70758 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70758/testReport)**
for PR 13077 at commit
GitHub user imatiach-msft opened a pull request:
https://github.com/apache/spark/pull/16441
[SPARK-14975][ML][WIP] Fixed GBTClassifier to predict probability per
training instance and fixed interfaces
## What changes were proposed in this pull request?
For all of the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13077
**[Test build #70758 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70758/testReport)**
for PR 13077 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16401#discussion_r94253091
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala
---
@@ -95,6 +96,29 @@ abstract class LogicalPlan
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16233
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70757/
Test PASSed.
---
Github user michaelkamprath commented on the issue:
https://github.com/apache/spark/pull/16403
@srowen To answer the use case question, it is primarily academic for
learning and testing. Students and researchers build clusters of Raspberry PI,
ODROID, or other SBCs to have a cost
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16233
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/16233
**[Test build #70757 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70757/testReport)**
for PR 16233 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16401#discussion_r94251780
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/estimation/EstimationSuite.scala
---
@@ -0,0 +1,67 @@
+/*
+ * Licensed to the Apache
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16401#discussion_r94251558
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -642,6 +642,13 @@ object SQLConf {
.doubleConf
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16401#discussion_r94251460
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala
---
@@ -95,6 +96,29 @@ abstract class LogicalPlan
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/15664
Thank you, @gatorsmile .
Happy New Year! :)
---
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16320
LGTM cc @cloud-fan
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/16403#discussion_r94249364
--- Diff: common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java
---
@@ -244,6 +251,18 @@ public static void throwException(Throwable t) {
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16404
LGTM cc @rxin
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so,
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15664
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 and
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/15664
---
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15664
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16371
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70756/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16371
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/16371
**[Test build #70756 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70756/testReport)**
for PR 16371 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16233
**[Test build #70757 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70757/testReport)**
for PR 16233 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15880
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70755/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15880
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/15880
**[Test build #70755 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70755/testReport)**
for PR 15880 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16404
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70754/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16404
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16404
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70753/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16404
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/16404
**[Test build #70753 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70753/testReport)**
for PR 16404 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16404
**[Test build #70754 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70754/testReport)**
for PR 16404 at commit
Github user cjuexuan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16428#discussion_r94239683
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -573,6 +573,7 @@ final class DataFrameWriter[T] private[sql](ds:
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16428#discussion_r94239452
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala ---
@@ -573,6 +573,7 @@ final class DataFrameWriter[T] private[sql](ds:
Github user cjuexuan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16428#discussion_r94239100
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVOptions.scala
---
@@ -71,7 +71,9 @@ private[csv] class
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/16428#discussion_r94238157
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVOptions.scala
---
@@ -71,7 +71,9 @@ private[csv] class
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16428
Ah, I meant to add a test there in this 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 not have this
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16417
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/16417
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70752/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16417
**[Test build #70752 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70752/testReport)**
for PR 16417 at commit
Github user cjuexuan commented on the issue:
https://github.com/apache/spark/pull/16428
@HyukjinKwon ,I already run `CSVSuite` ,and all tests 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16371
**[Test build #70756 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70756/testReport)**
for PR 16371 at commit
Github user cjuexuan commented on the issue:
https://github.com/apache/spark/pull/16428
@HyukjinKwon ,I see ,because my version is `2.0.2`,we use
`ByteArrayOutputStream` and call toString method ,this will using
`Charset.defaultCharset()` and bind with env ,and in master branch ,we
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16233#discussion_r94234598
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -510,32 +539,91 @@ class Analyzer(
* Replaces
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16371#discussion_r94234459
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/collect.scala
---
@@ -44,39 +44,48 @@ abstract class Collect
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16371#discussion_r94234334
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/RewriteDistinctAggregatesSuite.scala
---
@@ -63,7 +63,7 @@ class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16371#discussion_r94234295
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/RewriteDistinctAggregatesSuite.scala
---
@@ -63,7 +63,7 @@ class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16371#discussion_r94234184
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/RewriteDistinctAggregatesSuite.scala
---
@@ -63,7 +63,7 @@ class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16371#discussion_r94233910
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/RewriteDistinctAggregatesSuite.scala
---
@@ -63,7 +63,7 @@ class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16371#discussion_r94233881
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/RewriteDistinctAggregatesSuite.scala
---
@@ -63,7 +63,7 @@ class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16371#discussion_r94233778
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/RewriteDistinctAggregatesSuite.scala
---
@@ -63,7 +63,7 @@ class
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16371#discussion_r94232541
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/RewriteDistinctAggregatesSuite.scala
---
@@ -63,7 +63,7 @@ class
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16401
Just one minor question about the config. other 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
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/16401#discussion_r94232049
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -642,6 +642,13 @@ object SQLConf {
.doubleConf
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16233
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16233
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70750/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16233
**[Test build #70750 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70750/testReport)**
for PR 16233 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15880
**[Test build #70755 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70755/testReport)**
for PR 15880 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15880
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16401
LGTM. What if we just add the conf parameter to the `statistics` method and
give it a default value? e.g. `def statistics(conf: CatalystConf =
SimpleCatalystConf)`. How much mode do we need to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16404
**[Test build #70754 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70754/testReport)**
for PR 16404 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16404
**[Test build #70753 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70753/testReport)**
for PR 16404 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16404#discussion_r94229396
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1918,28 +1918,37 @@ class Analyzer(
case
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16404
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16233
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70748/
Test PASSed.
---
1 - 100 of 207 matches
Mail list logo