Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18718
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 so, or
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18718
@shaofei007 I actually think your fix is correct, so I wouldn't close it.
The comment says: ` /** Clear the old time-to-files mappings along with old
RDDs */` But only the superclass method clears
Github user shaofei007 commented on the issue:
https://github.com/apache/spark/pull/18718
@tdas
how do you think about this PR please?
@asfgit
Can we close it please?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18718
**[Test build #3848 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3848/testReport)**
for PR 18718 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18718
**[Test build #3848 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3848/testReport)**
for PR 18718 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/18718
Seems reasonable, but, I am sort of not sure why it hasn't come up before.
@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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18718
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