Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15769
**[Test build #68330 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68330/consoleFull)**
for PR 15769 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15047
@tejasapatil It sounds like the test case coverage is limited. It does not
cover all the data types, right?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/15769
@mridulm I've added most of your comments. Thank you!
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15805
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/15805
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68321/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15775
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/15805
**[Test build #68321 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68321/consoleFull)**
for PR 15805 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15775
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68320/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15775
**[Test build #68320 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68320/consoleFull)**
for PR 15775 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15798
SGTM. Thanks for doing this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/15637
Point query of countMinSketch can only help us on estimation for predicates
like col=1.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/15637
With (value, freq) pairs, i.e. equi-width histogram, given a predicate
col<100 and col>50, we can know exactly how many records are in that range,
which is the estimated rowcount after the filter.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15172
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/15172
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68319/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15172
**[Test build #68319 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68319/consoleFull)**
for PR 15172 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15797#discussion_r86936905
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -171,6 +171,10 @@ case class CatalogTable(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15797#discussion_r86936775
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/catalog/ExternalCatalogSuite.scala
---
@@ -591,25 +673,25 @@ abstract class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/15797#discussion_r86935972
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/PartitioningUtils.scala
---
@@ -451,51 +444,6 @@ object PartitioningUtils {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15742
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68318/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15742
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/15797
**[Test build #68329 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68329/consoleFull)**
for PR 15797 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15742
**[Test build #68318 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68318/consoleFull)**
for PR 15742 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15797
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15637
Weird - github didn't email me notifications.
Why do you care about all (value, freq) pairs? All you want should just be
some summary statistics that can tell you given a value, what the
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15805
LGTM pending Jenkins.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15805
**[Test build #68328 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68328/consoleFull)**
for PR 15805 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15797
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68322/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15797
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/15797
**[Test build #68322 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68322/consoleFull)**
for PR 15797 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15717
**[Test build #68327 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68327/consoleFull)**
for PR 15717 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68326 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68326/consoleFull)**
for PR 15751 at commit
Github user brkyvz commented on the issue:
https://github.com/apache/spark/pull/15806
Hi @oza, thank you for this patch, however I'm not sure if this patch fixes
anything. What was your problem, what didn't work for you?
---
If your project is set up for it, you can reply to this
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/15806#discussion_r86933542
--- Diff:
examples/src/main/scala/org/apache/spark/examples/sql/streaming/StructuredNetworkWordCount.scala
---
@@ -68,6 +70,8 @@ object
Github user brkyvz commented on a diff in the pull request:
https://github.com/apache/spark/pull/15806#discussion_r86933461
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/StreamingQueryManager.scala
---
@@ -219,10 +219,11 @@ class StreamingQueryManager
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15751
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/15804
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68317/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15804
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/15804
**[Test build #68317 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68317/consoleFull)**
for PR 15804 at commit
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/15047
@gatorsmile : I have tests in `HiveHasherSuite` to compare the values
against expected one. Initially I had thought about generating random input and
calling the original Hive's hash function
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/9
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/9
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68325/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9
**[Test build #68325 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68325/consoleFull)**
for PR 9 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/15805
LGTM except a minor comment and test pending Jenkins
---
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 a diff in the pull request:
https://github.com/apache/spark/pull/15805#discussion_r86932312
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -324,38 +324,45 @@ case class TruncateTableCommand(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9
**[Test build #68325 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68325/consoleFull)**
for PR 9 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15717
**[Test build #68324 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68324/consoleFull)**
for PR 15717 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15657
What's this about? Close it?
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/15805
LGTM otherwise.
---
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 weiqingy commented on a diff in the pull request:
https://github.com/apache/spark/pull/15563#discussion_r86930677
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2523,6 +2524,8 @@ private[spark] class CallerContext(
taskId: Option[Long] =
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15806
cc @brkyvz
---
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 ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/15627#discussion_r86930468
--- Diff: yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala ---
@@ -600,10 +600,14 @@ private[spark] class Client(
val (_,
Github user weiqingy commented on a diff in the pull request:
https://github.com/apache/spark/pull/15563#discussion_r86930281
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2551,6 +2551,7 @@ private[util] object CallerContext extends Logging {
*/
Github user weiqingy commented on a diff in the pull request:
https://github.com/apache/spark/pull/15563#discussion_r86930303
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2591,6 +2595,16 @@ private[spark] class CallerContext(
}
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15668
**[Test build #68323 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68323/consoleFull)**
for PR 15668 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/11105
Is there a way for this to have just one class and have both values
available always?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15746#discussion_r86929250
--- Diff: R/pkg/R/mllib.R ---
@@ -1863,5 +1889,209 @@ print.summary.RandomForestRegressionModel <-
function(x, ...) {
#' @export
#' @note
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15668
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/15668
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68315/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15668
**[Test build #68315 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68315/consoleFull)**
for PR 15668 at commit
Github user Yunni commented on the issue:
https://github.com/apache/spark/pull/15148
@jkbradley I agree with most of your comments above. And I would like to
suggest the following:
- I would recommend a more intuitive name like `HyperplaneProjection`
instead of `PStableHashing`
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/15746#discussion_r86928504
--- Diff: R/pkg/R/mllib.R ---
@@ -1828,13 +1849,13 @@ setMethod("summary", signature(object =
"RandomForestRegressionModel"),
#' @note
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/15790
opened SPARK-18347
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15797
**[Test build #68322 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68322/consoleFull)**
for PR 15797 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15806
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 oza opened a pull request:
https://github.com/apache/spark/pull/15806
[SPARK-18345][STRUCTURED STREAMING] Structured Streaming quick examples
fails with default configuration
## What changes were proposed in this pull request?
This PR fixes failure of a quick
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15805
**[Test build #68321 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68321/consoleFull)**
for PR 15805 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15805
cc @gatorsmile
---
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 cloud-fan opened a pull request:
https://github.com/apache/spark/pull/15805
[SPARK-18346][SQL] TRUNCATE TABLE should fail if no partition is matched
for the given non-partial partition spec
## What changes were proposed in this pull request?
a follow up of
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15775
**[Test build #68320 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68320/consoleFull)**
for PR 15775 at commit
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/15775
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/15172
**[Test build #68319 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68319/consoleFull)**
for PR 15172 at commit
Github user cjjnjust commented on the issue:
https://github.com/apache/spark/pull/15172
Build #68313 looks weird, details in link show it passed all test.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15742
**[Test build #68318 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68318/consoleFull)**
for PR 15742 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15797
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68314/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15797
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/15797
**[Test build #68314 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68314/consoleFull)**
for PR 15797 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68316/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68316 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68316/consoleFull)**
for PR 15751 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15751
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/15804
**[Test build #68317 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68317/consoleFull)**
for PR 15804 at commit
Github user brkyvz commented on the issue:
https://github.com/apache/spark/pull/15804
cc @tdas
---
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 brkyvz opened a pull request:
https://github.com/apache/spark/pull/15804
[SPARK-18342] Make rename failures fatal in HDFSBackedStateStore
## What changes were proposed in this pull request?
If the rename operation in the state store fails (`fs.rename` returns
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/15627
@kishorvpatil @tgravescs It seems this pr is breaking functionalities of
`--files` or `--archives`.
Using `--files` or `--archives` with files which are not included to
`--jars` doesn't work.
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15803
If I remember correctly, does not UI change require a screenshot in the PR
description?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15172
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/15172
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68311/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15172
**[Test build #68311 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68311/consoleFull)**
for PR 15172 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15172
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/15172
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68313/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15172
**[Test build #68313 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68313/consoleFull)**
for PR 15172 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15745
**[Test build #68310 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68310/consoleFull)**
for PR 15745 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15751
**[Test build #68316 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68316/consoleFull)**
for PR 15751 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15668
LGTM, pending jenkins
---
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/15779
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/15779
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/68312/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15779
**[Test build #68312 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68312/consoleFull)**
for PR 15779 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15668
**[Test build #68315 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/68315/consoleFull)**
for PR 15668 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/15668#discussion_r86919161
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/RewriteDistinctAggregates.scala
---
@@ -115,9 +115,22 @@ object
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/15802
Seems it does not work with sbt?
---
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 cloud-fan commented on the issue:
https://github.com/apache/spark/pull/15668
add to whitelist
---
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,
1 - 100 of 522 matches
Mail list logo