Github user dbolshak commented on a diff in the pull request:
https://github.com/apache/spark/pull/17458#discussion_r108678514
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllStagesPage.scala
---
@@ -34,9 +34,9 @@ private[ui] class AllStagesPage(parent: StagesTab)
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/12004
Any comments on the latest patch? Anyone?
---
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/17468
(BTW, I believe this does not make a conflict with PR 17406)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15049
**[Test build #75360 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75360/testReport)**
for PR 15049 at commit
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/14731
Is there anything else I need to do here?
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17468
**[Test build #75357 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75357/testReport)**
for PR 17468 at commit
Github user dbolshak commented on the issue:
https://github.com/apache/spark/pull/17458
Looks like all comments have been addressed.
---
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 opened a pull request:
https://github.com/apache/spark/pull/17468
[SPARK-20143][SQL] DataType.fromJson should throw an exception with better
message
## What changes were proposed in this pull request?
Currently, `DataType.fromJson` throws
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17417
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75355/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17417
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 squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/15332#discussion_r108682129
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java
---
@@ -362,7 +375,15 @@ private void
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/15332#discussion_r108681155
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/util/DateTimeUtils.scala
---
@@ -237,6 +238,24 @@ object DateTimeUtils {
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17435#discussion_r108685381
--- Diff: python/pyspark/sql/types.py ---
@@ -57,7 +57,25 @@ def __ne__(self, other):
@classmethod
def typeName(cls):
-
Github user squito commented on the issue:
https://github.com/apache/spark/pull/16781
thanks @ueshin ... I am going to chat w/ some folks involved in that hive
patch, that was not my understanding conceptually of their patch. I heard that
there is a bug they need to fix so maybe its
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17417
**[Test build #75355 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75355/testReport)**
for PR 17417 at commit
Github user szalai1 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17435#discussion_r108683954
--- Diff: python/pyspark/sql/types.py ---
@@ -57,7 +57,25 @@ def __ne__(self, other):
@classmethod
def typeName(cls):
-
Github user dbolshak commented on a diff in the pull request:
https://github.com/apache/spark/pull/17458#discussion_r10860
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala ---
@@ -290,7 +290,7 @@ private[ui] class StagePage(parent: StagesTab) extends
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/17364
I don't have a time/plans to do the test here, as it's a fairly complex
piece of test setup for what a review should show isn't doing anything other
than guarantee the outcome pf
Github user dbolshak commented on a diff in the pull request:
https://github.com/apache/spark/pull/17458#discussion_r108677907
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/StagesTab.scala ---
@@ -35,7 +35,7 @@ private[ui] class StagesTab(parent: SparkUI) extends
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17435#discussion_r108691360
--- Diff: python/pyspark/sql/types.py ---
@@ -57,7 +57,25 @@ def __ne__(self, other):
@classmethod
def typeName(cls):
-
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16611
**[Test build #75359 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75359/testReport)**
for PR 16611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16989
**[Test build #75358 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75358/testReport)**
for PR 16989 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15332
**[Test build #75361 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75361/testReport)**
for PR 15332 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16611
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/17388
---
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/17469
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13932
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75356/
Test PASSed.
---
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/17170#discussion_r108735289
--- Diff: R/pkg/R/mllib_fpm.R ---
@@ -99,7 +99,10 @@ setMethod("spark.fpGrowth", signature(data =
"SparkDataFrame"),
# Get frequent itemsets.
Github user shaneknapp commented on the issue:
https://github.com/apache/spark/pull/17355
lgtm++
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16989
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75358/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13932
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 dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/15332#discussion_r108726983
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/execution/datasources/parquet/VectorizedColumnReader.java
---
@@ -362,7 +375,15 @@ private void
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17468
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75357/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17468
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/16611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75359/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15049
**[Test build #75360 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75360/testReport)**
for PR 15049 at commit
GitHub user map222 opened a pull request:
https://github.com/apache/spark/pull/17469
[SPARK-20132][Docs]
## What changes were proposed in this pull request?
Add docstrings to column.py for the Column functions `rlike`, `like`,
`startswith`, and `endswith`. Pass these docstrings
Github user nsyca commented on a diff in the pull request:
https://github.com/apache/spark/pull/17450#discussion_r108720512
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala
---
@@ -384,33 +379,13 @@ case class NullPropagation(conf:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16611
**[Test build #75359 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75359/testReport)**
for PR 16611 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13932
**[Test build #75356 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75356/testReport)**
for PR 13932 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16989
**[Test build #75358 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75358/testReport)**
for PR 16989 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16989
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/15049
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75360/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15049
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 nsyca commented on a diff in the pull request:
https://github.com/apache/spark/pull/17450#discussion_r108720888
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -1122,7 +1119,7 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17468
**[Test build #75357 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75357/testReport)**
for PR 17468 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/17388
Merging to master / 2.1.
---
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 ron8hu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17415#discussion_r108754614
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -515,8 +530,138 @@ case
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17470
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/17419#discussion_r108743634
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/stat/SummarizerSuite.scala ---
@@ -335,4 +335,65 @@ class SummarizerSuite extends SparkFunSuite with
Github user kunalkhamar commented on the issue:
https://github.com/apache/spark/pull/17379
@hvanhovell it should be HiveSessionStateBuilder 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 does not
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17355
---
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/17470
**[Test build #75362 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75362/testReport)**
for PR 17470 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17470
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75362/
Test PASSed.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17251
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17251
**[Test build #75364 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75364/testReport)**
for PR 17251 at commit
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/17445
Have you seen #17088? I just glanced at this quickly but I think this is a
duplicate of that (SPARK-19753)
---
If your project is set up for it, you can reply to this email and have your
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17450#discussion_r108755645
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -1122,7 +1119,7 @@ case class
GitHub user sitalkedia opened a pull request:
https://github.com/apache/spark/pull/17471
[SPARK-3577] Report Spill size on disk for UnsafeExternalSorter
## What changes were proposed in this pull request?
Report Spill size on disk for UnsafeExternalSorter
## How
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17419#discussion_r108742186
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/stat/SummarizerSuite.scala ---
@@ -335,4 +335,65 @@ class SummarizerSuite extends SparkFunSuite with
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/17445
Jenkins this is OK to 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 does not have this feature
enabled
GitHub user bomeng opened a pull request:
https://github.com/apache/spark/pull/17470
[SPARK-20146][SQL] fix comment missing issue for thrift server
## What changes were proposed in this pull request?
The column comment was missing while constructing the Hive TableSchema.
Github user ron8hu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17415#discussion_r108753830
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -515,8 +530,138 @@ case
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/17355
Merged to master. Please do not backport.
---
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/17450#discussion_r108756250
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -297,8 +297,8 @@ case class Lower(child:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15332
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/15332
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75361/
Test PASSed.
---
Github user samelamin commented on the issue:
https://github.com/apache/spark/pull/14527
@cloud-fan by adding a new API do you mean move the logic into a new
method in the Dataset class?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user kunalkhamar commented on a diff in the pull request:
https://github.com/apache/spark/pull/17379#discussion_r108748031
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/internal/BaseSessionStateBuilder.scala
---
@@ -134,6 +135,14 @@ abstract class
Github user ron8hu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17415#discussion_r108751882
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -515,8 +530,138 @@ case
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17470
**[Test build #75362 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75362/testReport)**
for PR 17470 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/17355
Great, yay 2.6 deprecation adventures :)
---
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/17424
@viirya That is not only for document generator. It can help us
review/correct the output of `DESC FUNCTIONS EXTENDED`. You can find many
inconsistent parts we should correct when reading
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/15821#discussion_r108744852
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -2828,4 +2839,16 @@ class Dataset[T] private[sql](
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/15821#discussion_r108744456
--- Diff: python/pyspark/sql/tests.py ---
@@ -56,6 +56,15 @@
from pyspark.sql.utils import AnalysisException, ParseException,
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/15821#discussion_r108746678
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/Dataset.scala ---
@@ -2747,6 +2747,17 @@ class Dataset[T] private[sql](
}
}
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/17419#discussion_r108746608
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/stat/SummarizerSuite.scala ---
@@ -335,4 +335,65 @@ class SummarizerSuite extends SparkFunSuite with
Github user ron8hu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17415#discussion_r108752975
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -515,8 +530,138 @@ case
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17471
**[Test build #75363 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75363/testReport)**
for PR 17471 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15332
**[Test build #75361 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75361/testReport)**
for PR 15332 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17450#discussion_r108764164
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -1122,7 +1119,7 @@ case class
Github user samelamin commented on the issue:
https://github.com/apache/spark/pull/17472
@gatorsmile I think my Intellij has some annoying auto indenting changes.
Does spark have some project wide settings I can auto import?
---
If your project is set up for it, you can reply to
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17471
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75363/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17251
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/17251
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75364/
Test PASSed.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17251
Hi, @gatorsmile .
Could you review this PR when you have sometime?
---
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 samelamin commented on the issue:
https://github.com/apache/spark/pull/17472
ok @gatorsmile i set default indentation to 2, how does it look 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17436
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 ueshin opened a pull request:
https://github.com/apache/spark/pull/17473
[SPARK-19088][SQL] Fix 2.10 build.
## What changes were proposed in this pull request?
Commit 6c70a38 broke the build for scala 2.10. The commit uses some
reflections which are not
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17170
**[Test build #75366 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75366/testReport)**
for PR 17170 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17472#discussion_r108797960
--- Diff: common/unsafe/src/main/java/org/apache/spark/unsafe/Platform.java
---
@@ -46,18 +46,22 @@
private static final boolean unaligned;
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17449
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/17449
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17472
Clear code comments can help code reading. : )
---
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17419#discussion_r108838518
--- Diff: mllib/src/main/scala/org/apache/spark/ml/stat/Summarizer.scala ---
@@ -0,0 +1,746 @@
+/*
+ * Licensed to the Apache Software Foundation
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17251
Will review it tonight.
---
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 felixcheung commented on the issue:
https://github.com/apache/spark/pull/17474
merged to master, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17474
---
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 rxin commented on the issue:
https://github.com/apache/spark/pull/17476
cc @ericl, @bogdanrdc, @adrian-ionescu, @cloud-fan
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
GitHub user rxin opened a pull request:
https://github.com/apache/spark/pull/17476
[SPARK-20151][SQL] Account for partition pruning in scan metadataTime
metrics
## What changes were proposed in this pull request?
After SPARK-20136, we report metadata timing metrics in scan
1 - 100 of 353 matches
Mail list logo