Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16841
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72984/
Test PASSed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16841
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16949
**[Test build #72989 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/72989/testReport)**
for PR 16949 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16841
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16841
Thanks! 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
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16952
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72987/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16947
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16841#discussion_r101462455
--- Diff:
sql/core/src/test/resources/sql-tests/inputs/subquery/in-subquery/in-multiple-columns.sql
---
@@ -0,0 +1,127 @@
+-- A test suite for
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16947
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72988/
Test FAILed.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16841
---
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/16949
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/16952
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/16949
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/72986/
Test FAILed.
---
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101461466
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -296,6 +296,17 @@ object SQLConf {
.longConf
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101461106
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -296,6 +296,17 @@ object SQLConf {
.longConf
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101462262
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -161,23 +161,49 @@ private[hive] class
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r101461282
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -163,6 +163,10 @@ case class BucketSpec(
* @param
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/16949
terminated by signal 9. 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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r101640636
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,569 @@
+/*
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16964
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/16964
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73011/
Test FAILed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r101642131
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,569 @@
+/*
GitHub user Yunni opened a pull request:
https://github.com/apache/spark/pull/16966
[SPARK-18409][ML]LSH approxNearestNeighbors should use approxQuantile
instead of sort
## What changes were proposed in this pull request?
In previous implementation of LSH
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16966
**[Test build #73018 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73018/testReport)**
for PR 16966 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16965
**[Test build #73016 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73016/testReport)**
for PR 16965 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r101645748
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,569 @@
+/*
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15125
**[Test build #73019 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73019/testReport)**
for PR 15125 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r101650369
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,569 @@
+/*
Github user thunterdb commented on a diff in the pull request:
https://github.com/apache/spark/pull/15770#discussion_r101663790
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/clustering/PowerIterationClustering.scala
---
@@ -0,0 +1,182 @@
+/*
+ * Licensed to the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16969
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/16826
**[Test build #73027 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73027/testReport)**
for PR 16826 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15125
**[Test build #73019 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73019/testReport)**
for PR 15125 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15125
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/16970
**[Test build #73028 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73028/testReport)**
for PR 16970 at commit
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/16968
I see. I will drop the R example here, whichever PR goes in later can
finish the document update.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16826
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16826
**[Test build #73022 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73022/testReport)**
for PR 16826 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16826
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73022/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16951
**[Test build #73031 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73031/testReport)**
for PR 16951 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16386
**[Test build #73032 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73032/testReport)**
for PR 16386 at commit
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/16690
@srowen
How do you think about https://github.com/apache/spark/pull/16790?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user jinxing64 commented on the issue:
https://github.com/apache/spark/pull/16790
https://github.com/apache/spark/pull/16690#discussion_r101616883 causes the
build to produce lots of deprecation warnings.
@srowen @vanzin How do you think about this ?
---
If your project
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16826
**[Test build #73027 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73027/testReport)**
for PR 16826 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16826
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73027/
Test FAILed.
---
Github user sameeragarwal commented on a diff in the pull request:
https://github.com/apache/spark/pull/16499#discussion_r101674560
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1018,7 +1025,9 @@ private[spark] class BlockManager(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16826
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 zzcclp commented on the issue:
https://github.com/apache/spark/pull/16964
@srowen after update to master, in Eclipse IDE, there is an error in
JavaConsumerStrategySuite.java line 52:
`final Map offsets = new HashMap<>();
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/16499#discussion_r101675576
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -813,7 +813,14 @@ private[spark] class BlockManager(
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16851
gentle ping @srowen
---
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 thunterdb commented on a diff in the pull request:
https://github.com/apache/spark/pull/15770#discussion_r101664268
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/clustering/PowerIterationClustering.scala
---
@@ -0,0 +1,182 @@
+/*
+ * Licensed to the
Github user thunterdb commented on a diff in the pull request:
https://github.com/apache/spark/pull/15770#discussion_r101665899
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/clustering/PowerIterationClustering.scala
---
@@ -0,0 +1,182 @@
+/*
+ * Licensed to the
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/15125
LGTM. @felixcheung are we good to merge?
---
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 zsxwing opened a pull request:
https://github.com/apache/spark/pull/16970
[SPARK-19497][SS]Implement streaming deduplication
## What changes were proposed in this pull request?
This PR adds a special streaming deduplication operator to support
`dropDuplicates`
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16962
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/16962
**[Test build #73023 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73023/testReport)**
for PR 16962 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16962
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73023/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16386
**[Test build #73029 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73029/testReport)**
for PR 16386 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16386
**[Test build #73030 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73030/testReport)**
for PR 16386 at commit
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/16949
@vanzin I opened a jira (https://issues.apache.org/jira/browse/SPARK-19642)
to research and address the potential security flaws. Do you mind if I continue
this pr?
---
If your project is set up
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16951
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 vanzin commented on the issue:
https://github.com/apache/spark/pull/16949
Sure, this PR is fine, I'd just prefer some minor API adjustments to bring
it closer to the code I linked above.
---
If your project is set up for it, you can reply to this email and have your
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16785
> this looks like a very big hammer to solve this problem. Can't we try a
different approach?
I think we should try to avoid optimizing already optimized code snippets,
you might be able to do
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16785
**[Test build #73034 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73034/testReport)**
for PR 16785 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16971
**[Test build #73033 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73033/testReport)**
for PR 16971 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16969
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73025/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15125
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73019/
Test PASSed.
---
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16951#discussion_r101676617
--- Diff: R/pkg/R/stats.R ---
@@ -149,15 +149,18 @@ setMethod("freqItems", signature(x =
"SparkDataFrame", cols = "character"),
#' This method
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16951#discussion_r101676637
--- Diff: R/pkg/R/stats.R ---
@@ -149,15 +149,18 @@ setMethod("freqItems", signature(x =
"SparkDataFrame", cols = "character"),
#' This method
GitHub user zhengruifeng opened a pull request:
https://github.com/apache/spark/pull/16971
[SPARK-19573][SQL] Make NaN/null handling consistent in approxQuantile
## What changes were proposed in this pull request?
update `StatFunctions.multipleApproxQuantiles` to handle NaN/null
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16970
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73028/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16970
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 thunterdb commented on a diff in the pull request:
https://github.com/apache/spark/pull/15770#discussion_r101666251
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/clustering/PowerIterationClusteringSuite.scala
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to the
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16962
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16962#discussion_r101666288
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/SaveIntoDataSourceCommand.scala
---
@@ -0,0 +1,52 @@
+/*
+ *
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16611
Sure, I will rebase and update.
---
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16938
One more case:
5. `CREATE TABLE` or `CTAS` without the location spec: if the default path
exists, should we succeed or fail?
After we finishing the TABLE-level DDLs, we also need to
Github user NathanHowell commented on a diff in the pull request:
https://github.com/apache/spark/pull/16386#discussion_r101671453
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1764,4 +1769,117 @@ class JsonSuite extends
Github user NathanHowell commented on the issue:
https://github.com/apache/spark/pull/16386
@cloud-fan When implementing tests for the other modes I've uncovered an
existing bug in schema inference in `DROPMALFORMED` mode:
https://issues.apache.org/jira/browse/SPARK-19641. Since it
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/16923#discussion_r101678941
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -106,21 +106,33 @@ private[hive] class HiveClientImpl(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16970
**[Test build #73028 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73028/testReport)**
for PR 16970 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/16923
@vanzin , would you mind helping to review this PR, thanks a lot.
IIUC the issue was introduced in #11510 .
---
If your project is set up for it, you can reply to this email and have
Github user gczsjdy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16476#discussion_r101673768
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/conditionalExpressions.scala
---
@@ -340,3 +341,91 @@ object CaseKeyWhen {
Github user sameeragarwal commented on a diff in the pull request:
https://github.com/apache/spark/pull/16499#discussion_r101664624
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -813,7 +813,14 @@ private[spark] class BlockManager(
Github user mallman commented on a diff in the pull request:
https://github.com/apache/spark/pull/16499#discussion_r101675669
--- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala
---
@@ -1018,7 +1025,9 @@ private[spark] class BlockManager(
try {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16951
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73031/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16951
**[Test build #73031 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73031/testReport)**
for PR 16951 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16785
**[Test build #73035 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73035/testReport)**
for PR 16785 at commit
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/12524
Per [my comment on the
JIRA](https://issues.apache.org/jira/browse/SPARK-14658?focusedCommentId=15870777=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15870777),
I
GitHub user hhbyyh opened a pull request:
https://github.com/apache/spark/pull/16968
[SPARK-19337] [ML] [Dcoc] Documentation and examples for LinearSVC
## What changes were proposed in this pull request?
Documentation and examples (Java, scala, python, R) for LinearSVC
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16944
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73014/
Test PASSed.
---
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r101651211
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,569 @@
+/*
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r101653079
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,569 @@
+/*
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r101652923
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,569 @@
+/*
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r101657416
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,569 @@
+/*
+
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16928
definitely we should match the behavior of json
---
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16928
Thanks, let me review further within tomorrow.
---
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/16826
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/16826
**[Test build #73024 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73024/testReport)**
for PR 16826 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16826
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73024/
Test FAILed.
---
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r101659663
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,569 @@
+/*
+
1 - 100 of 547 matches
Mail list logo