Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64218 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64218/consoleFull)**
for PR 10896 at commit
[`86068d0`](https://github.com/apache/spark/commit/
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/14038
There's no performance problem from filtering just on names. It's when
people try to filter on more complex things (file type, timestamp) they need to
call `getFileStatus(path)` and that's the
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12889#discussion_r75735832
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -788,6 +788,8 @@ class GeneralizedLinearRegressionMode
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14759
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14759
**[Test build #64224 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64224/consoleFull)**
for PR 14759 at commit
[`349d95d`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14759
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64224/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14758
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64223/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14758
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14758
**[Test build #64223 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64223/consoleFull)**
for PR 14758 at commit
[`3ab82a0`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14625
**[Test build #64226 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64226/consoleFull)**
for PR 14625 at commit
[`e2677da`](https://github.com/apache/spark/commit/e
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14531#discussion_r75733463
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/tables.scala ---
@@ -75,18 +81,52 @@ case class CreateTableLikeCommand(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14572
**[Test build #64225 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64225/consoleFull)**
for PR 14572 at commit
[`a9c94ea`](https://github.com/apache/spark/commit/a
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14625
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/10896
@maropu this is shaping up nicely. I left a few more small comments.
---
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 projec
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/10896#discussion_r75732105
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/AggUtils.scala
---
@@ -27,26 +27,87 @@ import
org.apache.spark.sql.execution.
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14572
No problem. The conflicts are resolved. Thanks! @yhuai
---
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 ha
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/10896#discussion_r75731352
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/AggUtils.scala
---
@@ -27,26 +27,87 @@ import
org.apache.spark.sql.execution.
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14732
Nah it still has a link to the same document actually (which is why I
removed one of them), the only problem is I noticed the one I kept was a link
to an older version. It's not worth another change
Github user cnauroth commented on the issue:
https://github.com/apache/spark/pull/14659
+1 (non-binding) for the latest revision. I think this is an improvement,
speaking from the perspective of the HDFS audit log. Thank you, @Sherry302 .
:-)
---
If your project is set up for it,
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/14732
ah sorry... Feel free to send out a small pr to add the link back :)
---
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 doe
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14732#discussion_r75729969
--- Diff: docs/tuning.md ---
@@ -122,21 +122,8 @@ large records.
`R` is the storage space within `M` where cached blocks immune to being
evicted by execu
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14732
---
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 ena
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/14732
Oh heh, too late. No problem we may further improve the GC docs soon
anyway. The existing link wasn't wrong.
---
If your project is set up for it, you can reply to this email and have your
reply app
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14759
**[Test build #64224 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64224/consoleFull)**
for PR 14759 at commit
[`349d95d`](https://github.com/apache/spark/commit/3
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14759
cc @felixcheung @junyangq
---
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 wis
GitHub user shivaram opened a pull request:
https://github.com/apache/spark/pull/14759
[SPARK-16577][SPARKR] Add CRAN documentation checks to run-tests.sh
## What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
## How w
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14079
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64215/
Test PASSed.
---
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12889#discussion_r75729345
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -788,6 +788,8 @@ class GeneralizedLinearRegressionModel
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14079
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
e
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/14732
LGTM. Merging to master and branch 2.0. Thanks @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 th
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14079
**[Test build #64215 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64215/consoleFull)**
for PR 14079 at commit
[`fc45f5b`](https://github.com/apache/spark/commit/
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/12889#discussion_r75728898
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -788,6 +788,8 @@ class GeneralizedLinearRegressionMode
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14732#discussion_r75728831
--- Diff: docs/tuning.md ---
@@ -217,14 +204,22 @@ temporary objects created during task execution. Some
steps which may be useful
* Check if there are to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14758
**[Test build #64223 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64223/consoleFull)**
for PR 14758 at commit
[`3ab82a0`](https://github.com/apache/spark/commit/3
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/14732#discussion_r75727927
--- Diff: docs/tuning.md ---
@@ -217,14 +204,22 @@ temporary objects created during task execution. Some
steps which may be useful
* Check if there are t
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14735
Leaving it out of branch-2.0 sounds good to me.
---
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 f
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14758
cc @mengxr @felixcheung
FYI - This is mostly to ensure that we can have more maintainers who can
update the CRAN submissions. This shouldn't affect anything else on the
development side.
GitHub user shivaram opened a pull request:
https://github.com/apache/spark/pull/14758
[SPARKR][MINOR] Add Xiangrui and Felix to maintainers
## What changes were proposed in this pull request?
This change adds Xiangrui Meng and Felix Cheung to the maintainers field in
the p
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12889#discussion_r75727539
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -788,6 +788,8 @@ class GeneralizedLinearRegressionModel
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14753
**[Test build #64213 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64213/consoleFull)**
for PR 14753 at commit
[`10861b2`](https://github.com/apache/spark/commit/
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14735
I don't think they should be required for branch 2.0 - some part of the
signature change with ... is likely good to have for consistency but those
might also be "breaking" for a *.0.1 release
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14753
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14753
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64213/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14753
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14753
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64211/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14753
**[Test build #64211 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64211/consoleFull)**
for PR 14753 at commit
[`6efddad`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14757
**[Test build #64222 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64222/consoleFull)**
for PR 14757 at commit
[`f63826e`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14750
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64216/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14750
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14750
**[Test build #64216 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64216/consoleFull)**
for PR 14750 at commit
[`8fc6bcc`](https://github.com/apache/spark/commit/
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14732#discussion_r75722532
--- Diff: docs/tuning.md ---
@@ -122,21 +122,8 @@ large records.
`R` is the storage space within `M` where cached blocks immune to being
evicted by execut
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/14757
[SPARK-17190] [SQL] Removal of HiveSharedState
### What changes were proposed in this pull request?
Since `HiveClient` is used to interact with the Hive metastore, it should
be hidden in `Hi
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14732#discussion_r75722287
--- Diff: docs/tuning.md ---
@@ -217,14 +204,22 @@ temporary objects created during task execution. Some
steps which may be useful
* Check if there are to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14756
**[Test build #64221 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64221/consoleFull)**
for PR 14756 at commit
[`d600e68`](https://github.com/apache/spark/commit/d
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/14732#discussion_r75722253
--- Diff: docs/tuning.md ---
@@ -217,14 +204,22 @@ temporary objects created during task execution. Some
steps which may be useful
* Check if there are to
GitHub user clockfly opened a pull request:
https://github.com/apache/spark/pull/14756
[SPARK-17189][SQL][MINOR] Prefers InternalRow over UnsafeRow if UnsafeRow
specific interface is not used in AggregationIterator
## What changes were proposed in this pull request?
Minor c
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/14572
sorry. I missed this PR. Can you 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
enab
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14735
@felixcheung I didn't look at the code very closely, but will this change
be required in `branch-2.0` as well ? If so the merge might be hard to
---
If your project is set up for it, you can reply
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14734#discussion_r75720021
--- Diff: R/pkg/R/DataFrame.R ---
@@ -3058,7 +3057,7 @@ setMethod("str",
#' @note drop since 2.0.0
setMethod("drop",
signature(x = "
Github user junyangq commented on a diff in the pull request:
https://github.com/apache/spark/pull/14734#discussion_r75719798
--- Diff: R/pkg/NAMESPACE ---
@@ -1,5 +1,9 @@
# Imports from base R
-importFrom(methods, setGeneric, setMethod, setOldClass)
+# Do not include s
Github user junyangq commented on a diff in the pull request:
https://github.com/apache/spark/pull/14734#discussion_r75719536
--- Diff: R/pkg/R/DataFrame.R ---
@@ -3058,7 +3057,7 @@ setMethod("str",
#' @note drop since 2.0.0
setMethod("drop",
signature(x = "
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14734
LGTM. I had a couple of minor comments inline.
---
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 fe
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14734#discussion_r75718809
--- Diff: R/pkg/R/generics.R ---
@@ -1339,7 +1339,6 @@ setGeneric("spark.naiveBayes", function(data,
formula, ...) { standardGeneric("s
setGeneric("spa
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14734#discussion_r75718694
--- Diff: R/pkg/R/generics.R ---
@@ -1339,7 +1339,6 @@ setGeneric("spark.naiveBayes", function(data,
formula, ...) { standardGeneric("s
setGeneric("spa
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14734#discussion_r75718243
--- Diff: R/pkg/R/DataFrame.R ---
@@ -3058,7 +3057,7 @@ setMethod("str",
#' @note drop since 2.0.0
setMethod("drop",
signature(x = "
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14734#discussion_r75717898
--- Diff: R/pkg/NAMESPACE ---
@@ -1,5 +1,9 @@
# Imports from base R
-importFrom(methods, setGeneric, setMethod, setOldClass)
+# Do not include s
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14755
**[Test build #64219 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64219/consoleFull)**
for PR 14755 at commit
[`ea2d0cc`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14079
**[Test build #64220 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64220/consoleFull)**
for PR 14079 at commit
[`f8b1bff`](https://github.com/apache/spark/commit/f
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14743
---
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 ena
Github user junyangq commented on the issue:
https://github.com/apache/spark/pull/14735
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 feat
GitHub user clockfly opened a pull request:
https://github.com/apache/spark/pull/14755
[MINOR][SQL] Fix some typos in comments and test hints
## What changes were proposed in this pull request?
Fix some typos in comments and test hints
## How was this patch tested?
Github user squito commented on the issue:
https://github.com/apache/spark/pull/14079
also just realized that I forgot about @kayousterhout 's comment to add in
checks on the invariants for the confs -- I've added that now as well.
---
If your project is set up for it, you can reply
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14754
**[Test build #64217 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64217/consoleFull)**
for PR 14754 at commit
[`8ae3789`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64218 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64218/consoleFull)**
for PR 10896 at commit
[`86068d0`](https://github.com/apache/spark/commit/8
GitHub user clockfly opened a pull request:
https://github.com/apache/spark/pull/14754
[SPARK-17188][SQL] Moves class QuantileSummaries to project catalyst for
implementing percentile_approx
## What changes were proposed in this pull request?
This is a sub-task of SPARK-162
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14734
@junyangq Could you take one more look ? I will also do a pass now
---
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 do
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14743
BTW LGTM. Merging this PR into master, branch-2.0
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14743
Thanks @HyukjinKwon -- this is a bit surprising as it was only recently
that you fixed the windows tests in
https://github.com/apache/spark/commit/1c403733b89258e57daf7b8b0a2011981ad7ed8a
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8880#discussion_r75712664
--- Diff: core/src/main/scala/org/apache/spark/crypto/CryptoConf.scala ---
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) und
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8880#discussion_r75712428
--- Diff: core/src/main/scala/org/apache/spark/crypto/CryptoConf.scala ---
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) und
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64209/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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
e
Github user rajeshbalamohan commented on the issue:
https://github.com/apache/spark/pull/14537
For latest ORC, if the data was written out by Hive, it would have the same
mapping.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64209 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64209/consoleFull)**
for PR 10896 at commit
[`0375ac6`](https://github.com/apache/spark/commit/
Github user mallman commented on the issue:
https://github.com/apache/spark/pull/14537
@rajeshbalamohan So for Orc 2.x files, would schema inference be
unnecessary?
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12004
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64206/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/12004
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/12004
**[Test build #64206 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64206/consoleFull)**
for PR 12004 at commit
[`63cf84f`](https://github.com/apache/spark/commit/
Github user clockfly commented on a diff in the pull request:
https://github.com/apache/spark/pull/14298#discussion_r75709632
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/PercentileApprox.scala
---
@@ -0,0 +1,462 @@
+/*
+ * Lice
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14750
**[Test build #64216 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64216/consoleFull)**
for PR 14750 at commit
[`8fc6bcc`](https://github.com/apache/spark/commit/8
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/10896#discussion_r75707785
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/AggUtils.scala
---
@@ -27,26 +27,87 @@ import
org.apache.spark.sql.execution.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14239
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14239
**[Test build #64214 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64214/consoleFull)**
for PR 14239 at commit
[`c97c12f`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14239
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64214/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14079
**[Test build #64215 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64215/consoleFull)**
for PR 14079 at commit
[`fc45f5b`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14239
**[Test build #64214 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64214/consoleFull)**
for PR 14239 at commit
[`c97c12f`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14753
**[Test build #64213 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64213/consoleFull)**
for PR 14753 at commit
[`10861b2`](https://github.com/apache/spark/commit/1
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/14038
If my understanding is correct, `PathFilter` is not passed into
`FileSystem.listFiles` in `ListingFileCatalog#listLeafFiles` inside. If even
so, the performance degrades you pointed out occur?
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14239
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64212/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14239
**[Test build #64212 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64212/consoleFull)**
for PR 14239 at commit
[`b49be73`](https://github.com/apache/spark/commit/
401 - 500 of 757 matches
Mail list logo