Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16772
well...will merge after new tests
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16772
jenkins test 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
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/15821
## Future Work
The support of all Spark data types to include complex types will be a
quick follow up to ensure full compatibility. With the added ability to
convert a Dataset on the
Github user kayousterhout commented on a diff in the pull request:
https://github.com/apache/spark/pull/16930#discussion_r102863614
--- Diff: core/src/test/scala/org/apache/spark/executor/ExecutorSuite.scala
---
@@ -133,6 +122,79 @@ class ExecutorSuite extends SparkFunSuite {
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/16987
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17043
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/17043
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73380/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73380 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73380/testReport)**
for PR 17043 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16772
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16923
**[Test build #73386 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73386/testReport)**
for PR 16923 at commit
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17038
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
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17001#discussion_r102870703
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -408,7 +408,13 @@ private[spark] class
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/17001#discussion_r102871210
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -408,7 +408,13 @@ private[spark] class
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/16696
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, or
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/17001#discussion_r102873675
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -408,7 +408,13 @@ private[spark] class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17046
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 maropu commented on the issue:
https://github.com/apache/spark/pull/17046
@hvanhovell okay, ready to 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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73368 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73368/testReport)**
for PR 17043 at commit
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/12524
Can you update the PR description here to have the JIRA number
(SPARK-14658), not the PR number?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17009
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17044
**[Test build #73369 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73369/testReport)**
for PR 17044 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17041
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73358/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17041
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 shivaram commented on the issue:
https://github.com/apache/spark/pull/17018
Yeah asking them for 1.5hr seems like a good first step. We can also see
what fraction of time it takes for build vs. tests. I guess we can't make
builds faster without some effort, but we could
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/17043#discussion_r102843676
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaWriter.scala
---
@@ -0,0 +1,264 @@
+/*
+ * Licensed to the Apache
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17018
I see. Sure. Just as a side note, if you download the log, it prints the
times.
In this case, it was so close assuming from the last log :).
```
[00:59:47] Windows-specific
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17045
**[Test build #73370 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73370/testReport)**
for PR 17045 at commit
GitHub user mgummelt opened a pull request:
https://github.com/apache/spark/pull/17045
[SPARK-19373] Base spark.scheduler.minRegisteredResourceRatio on registered
cores râ¦
â¦ather than acquired cores in the Mesos Coarse Grained Scheduler
## What changes were proposed in
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17044
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73369/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17044
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 wangmiao1981 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17032#discussion_r102856602
--- Diff: examples/src/main/r/ml/kmeans.R ---
@@ -26,10 +26,12 @@ sparkR.session(appName = "SparkR-ML-kmeans-example")
# $example on$
#
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16992
**[Test build #73367 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73367/testReport)**
for PR 16992 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17032
**[Test build #73375 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73375/testReport)**
for PR 17032 at commit
Github user wangmiao1981 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17032#discussion_r102857809
--- Diff: R/pkg/vignettes/sparkr-vignettes.Rmd ---
@@ -565,11 +565,10 @@ We use a simple example to demonstrate `spark.logit`
usage. In general, there
Github user budde commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r102859179
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -161,22 +164,45 @@ private[hive] class
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/15415#discussion_r102860331
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/FPGrowth.scala ---
@@ -0,0 +1,346 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17046
**[Test build #73378 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73378/testReport)**
for PR 17046 at commit
Github user kayousterhout commented on a diff in the pull request:
https://github.com/apache/spark/pull/16855#discussion_r102848520
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala ---
@@ -664,6 +665,63 @@ class TaskSetManagerSuite extends
Github user kayousterhout commented on a diff in the pull request:
https://github.com/apache/spark/pull/16855#discussion_r102860963
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/TaskSetManagerSuite.scala ---
@@ -664,6 +665,55 @@ class TaskSetManagerSuite extends
Github user kunalkhamar commented on a diff in the pull request:
https://github.com/apache/spark/pull/16826#discussion_r102862437
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/SparkSession.scala
---
@@ -839,6 +869,7 @@ object SparkSession {
if
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73380 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73380/testReport)**
for PR 17043 at commit
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/16996#discussion_r102863165
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SharedState.scala ---
@@ -61,6 +62,9 @@ private[sql] class SharedState(val sparkContext:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73381 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73381/testReport)**
for PR 17043 at commit
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/12888
@hujy Thank you for sending this PR, and apologies for not seeing it
earlier. Since the other PR for this JIRA is ready to merge, could you please
close this issue? Thanks again!
---
If your
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/16534
Yes pydoc.help does depend on looking at the docstring on the type rather
than the object :( Too bad the IPython magic isn't used in pydoc too.
Sorry for all the back and forth, I'm just
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16996
**[Test build #73389 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73389/testReport)**
for PR 16996 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15415
**[Test build #73384 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73384/testReport)**
for PR 15415 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15415
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/15415
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73384/
Test PASSed.
---
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/17001#discussion_r102870072
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -408,7 +408,13 @@ private[spark] class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15125
**[Test build #73390 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73390/testReport)**
for PR 15125 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17001#discussion_r102870494
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -193,7 +193,16 @@ class SessionCatalog(
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/17001#discussion_r102870643
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -193,7 +193,16 @@ class SessionCatalog(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16395
**[Test build #73377 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73377/testReport)**
for PR 16395 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16395
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/16696
**[Test build #73391 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73391/testReport)**
for PR 16696 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16826
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/16826
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73379/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16923
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/16923
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73386/
Test PASSed.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17001
In `SharedState`, we should create the default database without location.
---
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/16923
**[Test build #73386 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73386/testReport)**
for PR 16923 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73368 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73368/testReport)**
for PR 17043 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16944
**[Test build #73361 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73361/testReport)**
for PR 16944 at commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/17044#discussion_r102838808
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamExecution.scala
---
@@ -709,12 +717,13 @@ class StreamExecution(
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r102843314
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -161,22 +164,45 @@ private[hive] class
Github user ericl commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r102841719
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -296,6 +296,25 @@ object SQLConf {
.longConf
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17042
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73362/
Test FAILed.
---
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/17043#discussion_r102846556
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaWriter.scala
---
@@ -0,0 +1,264 @@
+/*
+ * Licensed to the Apache
Github user budde commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r102850496
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -296,6 +296,25 @@ object SQLConf {
.longConf
Github user budde commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r102850475
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveMetastoreCatalog.scala ---
@@ -161,22 +164,45 @@ private[hive] class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16696
**[Test build #73372 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73372/testReport)**
for PR 16696 at commit
GitHub user lw-lin reopened a pull request:
https://github.com/apache/spark/pull/16987
[SPARK-19633][SS] FileSource read from FileSink
## What changes were proposed in this pull request?
Right now file source always uses `InMemoryFileIndex` to scan files from a
given path.
Github user lw-lin commented on the issue:
https://github.com/apache/spark/pull/16987
Reopening :-)
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16987
**[Test build #73374 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73374/testReport)**
for PR 16987 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16395
**[Test build #73376 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73376/testReport)**
for PR 16395 at commit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17033
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/16395
**[Test build #73377 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73377/testReport)**
for PR 16395 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17032
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73375/
Test PASSed.
---
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/15821
## Benchmarks with Conversion done on Executor (updated)
## 1mm Longs
_ | With Arrow | Without Arrow
--||---
count | 50.00 |50.00
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17032
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/17032
**[Test build #73375 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73375/testReport)**
for PR 17032 at commit
Github user kunalkhamar commented on a diff in the pull request:
https://github.com/apache/spark/pull/16826#discussion_r102861839
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -1178,4 +1178,36 @@ class SessionCatalog(
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16772
LGTM
Merging with master
@BryanCutler would you mind sending a backport PR against branch-2.1 to run
Jenkins tests?
Thank you!
---
If your project is set up for it, you can
GitHub user markgrover opened a pull request:
https://github.com/apache/spark/pull/17047
[SPARK-19720][SPARK SUBMIT] Redact sensitive information from SparkSubmit
console
## What changes were proposed in this pull request?
This change redacts senstive information (based on
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73381 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73381/testReport)**
for PR 17043 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17043
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73381/
Test PASSed.
---
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16772
OK merging now. @BryanCutler do let me know if you don't have time to send
a backport--thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16772
**[Test build #3581 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3581/testReport)**
for PR 16772 at commit
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/17001#discussion_r102865566
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -193,7 +193,16 @@ class SessionCatalog(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r102863871
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,511 @@
+/*
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16772
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r102862889
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,511 @@
+/*
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16772
**[Test build #73383 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73383/testReport)**
for PR 16772 at commit
Github user zjffdu commented on a diff in the pull request:
https://github.com/apache/spark/pull/16907#discussion_r102865692
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/api/r/SQLUtils.scala
---
@@ -48,13 +48,14 @@ private[sql] object SQLUtils extends Logging {
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r102860256
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,511 @@
+/*
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r102864512
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,511 @@
+/*
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r102860017
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -0,0 +1,511 @@
+/*
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16395#discussion_r102862837
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/Range.scala
---
@@ -57,6 +58,20 @@ object Range {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16772
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73383/
Test PASSed.
---
1 - 100 of 636 matches
Mail list logo