Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/17081#discussion_r104075579
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -106,6 +106,7 @@ case class DataSource(
*
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17061
Thank you! @vanzin
---
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17062
Will review it tonight. 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
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17081#discussion_r104073970
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -106,6 +106,7 @@ case class DataSource(
*
Github user uncleGen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17141#discussion_r104073813
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/ReservoirSampleSuit.scala
---
@@ -0,0 +1,134 @@
+/*
+ * Licensed to the Apache
Github user uncleGen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17141#discussion_r104073535
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/statefulOperators.scala
---
@@ -397,3 +402,110 @@ object
Github user uncleGen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17141#discussion_r104073674
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/statefulOperators.scala
---
@@ -397,3 +402,110 @@ object
Github user uncleGen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17141#discussion_r104073607
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/statefulOperators.scala
---
@@ -397,3 +402,110 @@ object
Github user uncleGen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17141#discussion_r104073516
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/statefulOperators.scala
---
@@ -397,3 +402,110 @@ object
Github user uncleGen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17141#discussion_r104073290
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/statefulOperators.scala
---
@@ -397,3 +402,110 @@ object
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17076
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
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17076
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73792/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17076
**[Test build #73792 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73792/testReport)**
for PR 17076 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/15666
It looks like a syntax error
```
inst/tests/testthat/test_context.R:170:79: error: unexpected '{'
test_that("add jar should work and allow usage of the jar on the driver
node" {
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17141
cc @zsxwing and @tdas
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17076
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73794/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17076
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/17076
**[Test build #73794 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73794/testReport)**
for PR 17076 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17080
**[Test build #73797 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73797/testReport)**
for PR 17080 at commit
Github user uncleGen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17080#discussion_r104070946
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala
---
@@ -82,17 +82,25 @@ class SparkHadoopUtil extends Logging {
//
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/17083
OK, Thanks @vanzin .
---
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 jerryshao closed the pull request at:
https://github.com/apache/spark/pull/17083
---
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 vanzin commented on the issue:
https://github.com/apache/spark/pull/17083
@jerryshao I think you'll need to manually close the 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
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16867
Also, thanks for doing the timing measurements!
---
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 vanzin commented on the issue:
https://github.com/apache/spark/pull/17083
Merging to 2.1 / 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 feature
enabled and wishes
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17083#discussion_r104070712
--- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala ---
@@ -330,7 +330,7 @@ private[spark] object JettyUtils extends Logging {
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16867
I'm a little on the fence about this because of the added complexity, but
it does seem to be a significant time improvement. Did you consider
implementing this as a median heap (see the last
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17141
@srowen There are some unsupported `operator` for Structured Streaming. You
can view here:
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/16970
@zsxwing Thanks, I am missing it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17081
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/17081
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73791/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17081
**[Test build #73791 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73791/testReport)**
for PR 17081 at commit
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/15505
@witgo I don't think the ~1.5% improvement in runtime merits the added
complexity of this change. I could be convinced to merge this if it simplified
the code or the ability to reason about
Github user lw-lin commented on the issue:
https://github.com/apache/spark/pull/16970
@uncleGen I think `requiredChildDistribution =
ClusteredDistribution(keyExpressions) :: Nil` (please see
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17074
**[Test build #73796 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73796/testReport)**
for PR 17074 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17074
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73796/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17074
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/16639
---
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 imatiach-msft commented on the issue:
https://github.com/apache/spark/pull/17076
I took another brief look, the updates look great, thank you
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17074
**[Test build #73796 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73796/testReport)**
for PR 17074 at commit
Github user jinxing64 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17133#discussion_r104066996
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala ---
@@ -695,7 +695,8 @@ private[spark] class TaskSetManager(
def
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16639
I merged this into master. Thanks @squito!
---
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 vanzin commented on the issue:
https://github.com/apache/spark/pull/17074
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17133
**[Test build #73795 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73795/testReport)**
for PR 17133 at commit
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/17076
Updated. Well I was able to verify that adding the lazy val does make the
task binary broadcast variable smaller - i.e. it does not vary with the size of
the features. We could potentially test this
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17076
**[Test build #73794 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73794/testReport)**
for PR 17076 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17081
**[Test build #73793 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73793/testReport)**
for PR 17081 at commit
Github user windpiger commented on a diff in the pull request:
https://github.com/apache/spark/pull/17081#discussion_r104065615
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -109,7 +109,9 @@ case class DataSource(
*
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17076
**[Test build #73792 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73792/testReport)**
for PR 17076 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/17061
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 jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/17083#discussion_r104064675
--- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala ---
@@ -330,7 +330,7 @@ private[spark] object JettyUtils extends Logging {
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17081
Please remove `[FOLLOW-UP]` from the PR title. 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
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17142#discussion_r104063545
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/UnivocityParser.scala
---
@@ -54,39 +54,77 @@ private[csv] class
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17081#discussion_r104063471
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -109,7 +109,9 @@ case class DataSource(
*
Github user kayousterhout commented on a diff in the pull request:
https://github.com/apache/spark/pull/17140#discussion_r104063441
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskDescription.scala ---
@@ -86,7 +86,10 @@ private[spark] object TaskDescription {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/16954#discussion_r104063308
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -21,12 +21,13 @@ import scala.annotation.tailrec
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17080#discussion_r104063068
--- Diff: core/src/main/scala/org/apache/spark/deploy/SparkHadoopUtil.scala
---
@@ -82,17 +82,25 @@ class SparkHadoopUtil extends Logging {
// the
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17142#discussion_r104062715
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/UnivocityParser.scala
---
@@ -54,39 +54,77 @@ private[csv] class
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17140#discussion_r104062543
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskDescription.scala ---
@@ -86,7 +86,10 @@ private[spark] object TaskDescription {
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16959
---
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 kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16959
LGTM -- I merged this into 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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17143
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73788/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17143
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/17143
**[Test build #73788 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73788/testReport)**
for PR 17143 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16954
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/16954
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73789/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16954
**[Test build #73789 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73789/testReport)**
for PR 16954 at commit
Github user windpiger commented on the issue:
https://github.com/apache/spark/pull/17081
oh...sorry ï¼ I don't know when I close it...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17130
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73790/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17130
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/17130
**[Test build #73790 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73790/testReport)**
for PR 17130 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17081
**[Test build #73791 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73791/testReport)**
for PR 17081 at commit
GitHub user windpiger reopened a pull request:
https://github.com/apache/spark/pull/17081
[SPARK-18726][SQL][FOLLOW-UP]resolveRelation for FileFormat DataSource
don't need to listFiles twice
## What changes were proposed in this pull request?
Currently when we
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/11601
Thanks a lot for making a pass @MLnick. The last update mainly focus on the
interface and behavior change. I'll make a pass and also address your comments.
---
If your project is set up for it,
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/17130
ping @jkbradley since we're changing the FPGrowth `transform`.
Sean made a great suggestion to simplify `transform` code.
---
If your project is set up for it, you can reply to this email and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17130
**[Test build #73790 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73790/testReport)**
for PR 17130 at commit
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/15666
So now the R style checks out of alignment (sorry) - if you have a chance
to look at those it would be great.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17130#discussion_r104039923
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/FPGrowthExample.scala ---
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/17044
@zsxwing I'm not the right person to review this, but can we get the ball
rolling on this fix? This test is really flaky and constantly causes PR
builders to fail.
---
If your project is set up
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16954
**[Test build #73789 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73789/testReport)**
for PR 16954 at commit
Github user sueann commented on a diff in the pull request:
https://github.com/apache/spark/pull/17090#discussion_r104036563
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/recommendation/ALSSuite.scala ---
@@ -594,6 +595,95 @@ class ALSSuite
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/17130#discussion_r104036137
--- Diff: mllib/src/main/scala/org/apache/spark/ml/fpm/FPGrowth.scala ---
@@ -56,8 +56,8 @@ private[fpm] trait FPGrowthParams extends Params with
Github user ash211 commented on the issue:
https://github.com/apache/spark/pull/16959
Any last changes before merging?
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17143
**[Test build #73788 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73788/testReport)**
for PR 17143 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17143
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/17143
The only failure is irrelevant to this PR.
```
[info] KafkaSourceStressForDontFailOnDataLossSuite:
[info] - stress test for failOnDataLoss=false *** FAILED *** (1 minute, 2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17143
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/17143
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73785/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17143
**[Test build #73785 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73785/testReport)**
for PR 17143 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/17091#discussion_r104028425
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala
---
@@ -222,12 +222,17 @@ class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17090
**[Test build #73787 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73787/testReport)**
for PR 17090 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17090
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/17090
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73787/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16639
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73784/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16639
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/16639
**[Test build #73784 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73784/testReport)**
for PR 16639 at commit
Github user kayousterhout commented on a diff in the pull request:
https://github.com/apache/spark/pull/17140#discussion_r104018967
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskDescription.scala ---
@@ -86,7 +86,10 @@ private[spark] object TaskDescription {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17090
**[Test build #73787 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73787/testReport)**
for PR 17090 at commit
Github user kayousterhout commented on a diff in the pull request:
https://github.com/apache/spark/pull/17140#discussion_r104016148
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskDescription.scala ---
@@ -86,7 +86,10 @@ private[spark] object TaskDescription {
Github user kayousterhout commented on a diff in the pull request:
https://github.com/apache/spark/pull/17140#discussion_r104015455
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/TaskDescription.scala ---
@@ -86,7 +86,10 @@ private[spark] object TaskDescription {
201 - 300 of 671 matches
Mail list logo