Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18920#discussion_r133002116
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameFunctionsSuite.scala ---
@@ -449,6 +451,49 @@ class DataFrameFunctionsSuite extends
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r133001818
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/ApplicationHistoryProvider.scala
---
@@ -76,6 +76,14 @@ private[history] case class
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r133001417
--- Diff: core/src/main/scala/org/apache/spark/status/api/v1/api.scala ---
@@ -31,6 +33,9 @@ class ApplicationInfo private[spark](
val
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r133001335
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -742,53 +698,145 @@ private[history] object FsHistoryProvider {
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/18887#discussion_r133000455
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/ApplicationHistoryProvider.scala
---
@@ -76,6 +76,14 @@ private[history] case class
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18918
Yes. We should fix it in `object PhysicalOperation`
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/18920
---
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/18920
Thanks! Merged 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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18920
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/18914
---
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 dbolshak commented on the issue:
https://github.com/apache/spark/pull/18940
LGTM, btw, no unit tests for the change?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/18929
They're there mainly to declare the constant as a public API that must not
change. (I'm not sure whether mima captures changes in constant values, since
that's a binary breaking change, but that's
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18914
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18700#discussion_r132996396
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -1096,8 +1099,42 @@ class SessionCatalog(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18700#discussion_r132994080
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -1096,8 +1099,42 @@ class SessionCatalog(
Github user byakuinss commented on the issue:
https://github.com/apache/spark/pull/18895
Okay, I leave a comment in the issue page.
---
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/9518
**[Test build #80638 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80638/testReport)**
for PR 9518 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18866
**[Test build #80637 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80637/testReport)**
for PR 18866 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18895
Can you maybe leave any comment saying.. like .. "here is my JIRA account."
in https://issues.apache.org/jira/browse/SPARK-21658 if you don't mind?
---
If your project is set up for it, you
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18895
Hm.. weird. I can't search your account on JIRA ...
---
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/18907
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/18907
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80630/
Test FAILed.
---
Github user byakuinss commented on the issue:
https://github.com/apache/spark/pull/18895
@HyukjinKwon
Oh, do you mean my jira full name? It's `Chin Han Yu`.
---
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/18907
**[Test build #80630 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80630/testReport)**
for PR 18907 at commit
Github user nchammas commented on the issue:
https://github.com/apache/spark/pull/18926
To summarize the feedback from @HyukjinKwon and @gatorsmile, I think what I
need to do is:
* Add a test for the mixed type case.
* Explicitly check for `long` in Python 2 and throw a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18940
**[Test build #80636 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80636/testReport)**
for PR 18940 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18468
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80629/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18468
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/18895
---
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18895
@byakuinss, BTW, do you mind if I ask your JIRA id? I want to assign this
to you as you resolved this but I can't find the ID..
---
If your project is set up for it, you can reply to this
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/18940
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 and wishes so, or if
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18895
Merged 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 wishes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18468
**[Test build #80629 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80629/testReport)**
for PR 18468 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18938
**[Test build #80635 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80635/testReport)**
for PR 18938 at commit
Github user jmchung commented on a diff in the pull request:
https://github.com/apache/spark/pull/18930#discussion_r132984129
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/jsonExpressions.scala
---
@@ -361,10 +361,18 @@ case class
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/18488#discussion_r132983668
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/ExpressionInfo.java
---
@@ -79,7 +79,7 @@ public ExpressionInfo(
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/18855#discussion_r132982799
--- Diff: project/SparkBuild.scala ---
@@ -790,7 +790,7 @@ object TestSettings {
javaOptions in Test ++=
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18930
**[Test build #80634 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80634/testReport)**
for PR 18930 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18940
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 redsanket opened a pull request:
https://github.com/apache/spark/pull/18940
YSPARK-734 Change CacheLoader to limit entries based on memory footprint
Right now the spark shuffle service has a cache for index files. It is
based on a # of files cached
Github user eyalfa commented on a diff in the pull request:
https://github.com/apache/spark/pull/18855#discussion_r132978316
--- Diff: project/SparkBuild.scala ---
@@ -790,7 +790,7 @@ object TestSettings {
javaOptions in Test ++=
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18895
LGTM too.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18895
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/18895
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80633/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18895
**[Test build #80633 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80633/testReport)**
for PR 18895 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18913
LGTM, merging to master/2.2!
---
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 viirya commented on the issue:
https://github.com/apache/spark/pull/18895
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/18913
---
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 mike0sv commented on the issue:
https://github.com/apache/spark/pull/18488
@srowen @HyukjinKwon it seems like it's all ok 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 have this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18895
**[Test build #80633 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80633/testReport)**
for PR 18895 at commit
Github user byakuinss commented on a diff in the pull request:
https://github.com/apache/spark/pull/18895#discussion_r132968529
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1403,6 +1403,16 @@ def replace(self, to_replace, value=None,
subset=None):
|null|
Github user byakuinss commented on a diff in the pull request:
https://github.com/apache/spark/pull/18895#discussion_r132968408
--- Diff: python/pyspark/sql/dataframe.py ---
@@ -1837,8 +1847,8 @@ def fill(self, value, subset=None):
fill.__doc__ =
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18895
**[Test build #80632 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80632/testReport)**
for PR 18895 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18895
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80632/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18895
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/18895
**[Test build #80632 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80632/testReport)**
for PR 18895 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18939
**[Test build #80631 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80631/testReport)**
for PR 18939 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18939
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18855#discussion_r132964212
--- Diff: project/SparkBuild.scala ---
@@ -790,7 +790,7 @@ object TestSettings {
javaOptions in Test ++=
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18907
**[Test build #80630 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80630/testReport)**
for PR 18907 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18700#discussion_r132961933
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -1096,8 +1099,42 @@ class SessionCatalog(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/18700#discussion_r132961262
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -1096,8 +1099,42 @@ class SessionCatalog(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18939
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80626/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18939
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/18939
**[Test build #80626 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80626/testReport)**
for PR 18939 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80628/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
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/18918
**[Test build #80628 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80628/testReport)**
for PR 18918 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18920
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80625/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18920
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/18920
**[Test build #80625 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80625/testReport)**
for PR 18920 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18934
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80627/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18934
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/18934
**[Test build #80627 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80627/testReport)**
for PR 18934 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18468
**[Test build #80629 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80629/testReport)**
for PR 18468 at commit
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18468
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18938
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80624/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18938
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/18938
**[Test build #80624 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80624/testReport)**
for PR 18938 at commit
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18902#discussion_r132939361
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Imputer.scala ---
@@ -133,23 +134,29 @@ class Imputer @Since("2.2.0") (@Since("2.2.0")
override
Github user MLnick commented on a diff in the pull request:
https://github.com/apache/spark/pull/18902#discussion_r132939323
--- Diff: mllib/src/main/scala/org/apache/spark/ml/feature/Imputer.scala ---
@@ -133,23 +134,29 @@ class Imputer @Since("2.2.0") (@Since("2.2.0")
override
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18929
Maybe @vanzin can weigh in here, because the real question is whether these
constants in the launcher module are meant to be _the_ single definition of
them used throughout the code. core depends on
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18920
**[Test build #80625 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80625/testReport)**
for PR 18920 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18939
**[Test build #80626 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80626/testReport)**
for PR 18939 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18918
**[Test build #80628 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80628/testReport)**
for PR 18918 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18934
**[Test build #80627 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80627/testReport)**
for PR 18934 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18938
**[Test build #80624 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80624/testReport)**
for PR 18938 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18937
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/18937
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80623/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18937
**[Test build #80623 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80623/testReport)**
for PR 18937 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18933
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/18933
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80622/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18933
**[Test build #80622 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80622/testReport)**
for PR 18933 at commit
Github user skonto commented on the issue:
https://github.com/apache/spark/pull/18630
@vanzin I dont forgot I didnt see agreement.
---
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/18935
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/18935
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80618/
Test FAILed.
---
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18918#discussion_r132921051
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -522,6 +522,8 @@ object ColumnPruning extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18935
**[Test build #80618 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80618/testReport)**
for PR 18935 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18918#discussion_r132919858
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/ColumnPruningSuite.scala
---
@@ -360,5 +360,34 @@ class ColumnPruningSuite
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/18929
@srowen @jerryshao
---
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
201 - 300 of 426 matches
Mail list logo