Github user lvdongr closed the pull request at:
https://github.com/apache/spark/pull/18751
---
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 WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/18610#discussion_r130015435
--- Diff: mllib/src/main/scala/org/apache/spark/ml/util/ReadWrite.scala ---
@@ -309,6 +313,23 @@ private[ml] object DefaultParamsWriter {
val
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18754
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/18754
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80014/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18754
**[Test build #80014 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80014/testReport)**
for PR 18754 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18540
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80012/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18540
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/18540
**[Test build #80012 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80012/testReport)**
for PR 18540 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18749
Will address comments within few days. (I am reading docs just to get used
to things around my updated status)
---
If your project is set up for it, you can reply to this email and have your
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18695
The change LGTM. Will it be hard to add a reliable test for 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
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/18655
@BryanCutler @wesm @cpcloud I filed a JIRA issue for decimal type support
[SPARK-21552](https://issues.apache.org/jira/browse/SPARK-21552) and sent a pr
for it as WIP #18754.
Let's move on there
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18755
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/18754
**[Test build #80014 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80014/testReport)**
for PR 18754 at commit
GitHub user davidxdh opened a pull request:
https://github.com/apache/spark/pull/18755
[SPARK-21553][Spark Shell] Added the description of the default value of
master parameter in the spark-shell
When I type spark-shell --help, I find that the default value description
for the
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/18754
[WIP][SPARK-21552][SQL] Add DecimalType support to ArrowWriter.
## What changes were proposed in this pull request?
Decimal type is not yet supported in `ArrowWriter`.
This is adding
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/17435
@szalai1 Could you fix tests if you're still working on 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17435
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/17435
**[Test build #80013 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80013/testReport)**
for PR 17435 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17435
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80013/
Test FAILed.
---
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/18185
@gatorsmile ok
---
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/18185
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80011/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18185
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/18185
**[Test build #80011 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80011/testReport)**
for PR 18185 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17435
**[Test build #80013 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80013/testReport)**
for PR 17435 at commit
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/17435
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 the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18753
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 lvdongr opened a pull request:
https://github.com/apache/spark/pull/18753
[SPARK-21548] [SQL] Support insert into serial columns of table
## What changes were proposed in this pull request?
When we use the 'insert into ...' statement we can only insert all the
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/18468
@cloud-fan how will we go forward? @rxin seems to have no comment for 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 viirya commented on the issue:
https://github.com/apache/spark/pull/18725
Can the current fix work for the case like the following?
Project [a]
Filter [rand() > 1]
TableScan [a, b, c]
`PhysicalOperation` still fails for
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/18525
@ajbozarth Okay.Thanks.
@srowen Help review the code.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
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18725#discussion_r13082
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -495,6 +495,23 @@ object ColumnPruning extends
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/18554
---
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 yanboliang commented on the issue:
https://github.com/apache/spark/pull/18554
Merged into master, thanks for all.
---
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/18540
**[Test build #80012 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80012/testReport)**
for PR 18540 at commit
Github user lvdongr closed the pull request at:
https://github.com/apache/spark/pull/17203
---
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 yaooqinn commented on a diff in the pull request:
https://github.com/apache/spark/pull/18668#discussion_r129998159
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveUtilsSuite.scala ---
@@ -33,4 +33,13 @@ class HiveUtilsSuite extends QueryTest with
Github user yanboliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/18610#discussion_r129997548
--- Diff: mllib/src/main/scala/org/apache/spark/ml/util/ReadWrite.scala ---
@@ -309,6 +313,23 @@ private[ml] object DefaultParamsWriter {
val
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18725#discussion_r129997298
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -495,6 +495,23 @@ object ColumnPruning extends
Github user heary-cao commented on a diff in the pull request:
https://github.com/apache/spark/pull/18725#discussion_r129997034
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -495,6 +495,23 @@ object ColumnPruning extends
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/16648
ping @bdrillard for the 2nd part of this PR
---
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/18185
**[Test build #80011 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80011/testReport)**
for PR 18185 at commit
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r129989469
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/param/shared/HasParallelism.scala ---
@@ -0,0 +1,56 @@
+/*
+ * Licensed to the Apache Software
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r129989379
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/OneVsRest.scala ---
@@ -294,6 +296,18 @@ final class OneVsRest @Since("1.4.0") (
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r129989332
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/OneVsRest.scala ---
@@ -294,6 +296,18 @@ final class OneVsRest @Since("1.4.0") (
Github user caneGuy commented on the issue:
https://github.com/apache/spark/pull/18745
cc @srowen Test done,any more problem?Thanks too much.
---
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 jkbradley commented on the issue:
https://github.com/apache/spark/pull/18281
@holdenk Some of those improvements on handling parallelism sounds useful,
but I'd prefer we merge this and then add more improvements. This PR should be
a strict improvement there (moving from
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18185
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/18185
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80010/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18185
**[Test build #80010 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80010/testReport)**
for PR 18185 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18185
**[Test build #80010 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80010/testReport)**
for PR 18185 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/18185
Thanks! Fixed.
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/18740
---
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/18740
Thanks! 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
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/18725#discussion_r129984515
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -495,6 +495,23 @@ object ColumnPruning extends
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/18281#discussion_r129982672
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/OneVsRestSuite.scala ---
@@ -101,6 +101,45 @@ class OneVsRestSuite extends
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/18652
@gatorsmile Actually it is not rare we add a feature step by step in
SparkSQL. This is not a reason preventing us from adding this support. I think
this change already help much this kind of
Github user zhzhan commented on the issue:
https://github.com/apache/spark/pull/17180
retest it 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 rxin commented on the issue:
https://github.com/apache/spark/pull/18752
cc @JoshRosen
---
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/18752
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 peay opened a pull request:
https://github.com/apache/spark/pull/18752
[SPARK-21551][Python] Increase timeout for PythonRDD.serveIterator
## What changes were proposed in this pull request?
This modification increases the timeout for `serveIterator` (which is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18740
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/18740
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80009/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18740
**[Test build #80009 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80009/testReport)**
for PR 18740 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18664
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80008/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18664
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/18664
**[Test build #80008 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80008/testReport)**
for PR 18664 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18555#discussion_r129946652
--- Diff: core/src/test/scala/org/apache/spark/SparkConfSuite.scala ---
@@ -322,6 +324,291 @@ class SparkConfSuite extends SparkFunSuite with
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18185
LGTM except a few minor comments. Thanks for working on 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
Github user gslowikowski commented on the issue:
https://github.com/apache/spark/pull/18750
Updated.
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18185#discussion_r129944781
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -859,6 +859,22 @@ class Analyzer(
//
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18185#discussion_r129944055
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
---
@@ -423,6 +423,26 @@ case class UnresolvedAlias(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18185#discussion_r129943797
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
---
@@ -423,6 +423,26 @@ case class UnresolvedAlias(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18185#discussion_r129943216
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/unresolved.scala
---
@@ -423,6 +423,26 @@ case class UnresolvedAlias(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18185#discussion_r129943038
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -750,20 +750,28 @@ class AstBuilder(conf: SQLConf)
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18740
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18740
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/18740
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80007/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18740
**[Test build #80007 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80007/testReport)**
for PR 18740 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/18749#discussion_r129927520
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/ExpressionInfo.java
---
@@ -26,6 +26,10 @@
private String name;
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/18749#discussion_r129929153
--- Diff:
sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expressions/ExpressionDescription.java
---
@@ -29,15 +29,40 @@
* show the usage of
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/18659#discussion_r129928163
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/arrow/ArrowConverters.scala
---
@@ -132,6 +135,61 @@ private[sql] object
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18740
**[Test build #80009 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80009/testReport)**
for PR 18740 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18664
**[Test build #80008 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80008/testReport)**
for PR 18664 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18749
cc @rxin, @srowen and @cloud-fan, I believe this one is ready for a review.
Could you take a look when you have some time?
---
If your project is set up for it, you can reply to this email and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18540
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/18540
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80006/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18540
**[Test build #80006 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80006/testReport)**
for PR 18540 at commit
Github user aokolnychyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/18740#discussion_r129911780
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -1304,6 +1304,15 @@ class DatasetSuite extends QueryTest with
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/18740#discussion_r129906375
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DatasetSuite.scala
---
@@ -1304,6 +1304,15 @@ class DatasetSuite extends QueryTest with
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18745
**[Test build #3853 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3853/testReport)**
for PR 18745 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18740
**[Test build #80007 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80007/testReport)**
for PR 18740 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18749
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/18749
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80004/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18749
**[Test build #80004 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80004/testReport)**
for PR 18749 at commit
Github user abellina commented on a diff in the pull request:
https://github.com/apache/spark/pull/18253#discussion_r129669738
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/EventLoggingListener.scala ---
@@ -131,94 +132,35 @@ private[spark] class EventLoggingListener(
Github user abellina commented on a diff in the pull request:
https://github.com/apache/spark/pull/18253#discussion_r129889123
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/SparkListenerBus.scala ---
@@ -27,7 +27,12 @@ private[spark] trait SparkListenerBus
Github user abellina commented on a diff in the pull request:
https://github.com/apache/spark/pull/18253#discussion_r129659027
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -2379,8 +2382,13 @@ class SparkContext(config: SparkConf) extends
Logging {
Github user abellina commented on a diff in the pull request:
https://github.com/apache/spark/pull/18253#discussion_r129886931
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/LiveListenerBus.scala ---
@@ -139,30 +148,60 @@ private[spark] class LiveListenerBus(conf:
Github user abellina commented on a diff in the pull request:
https://github.com/apache/spark/pull/18253#discussion_r129879433
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/LiveListenerBus.scala ---
@@ -39,98 +39,107 @@ import org.apache.spark.util.Utils
* has
Github user abellina commented on a diff in the pull request:
https://github.com/apache/spark/pull/18253#discussion_r129659490
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -532,7 +533,10 @@ class SparkContext(config: SparkConf) extends Logging {
1 - 100 of 244 matches
Mail list logo
