Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/12327#issuecomment-222195981
`cacheSize1` and `cacheSize2` should be almost same, because they cache the
same data, with different RDD partition numbers. However, before this PR,
`cacheSize1` is much larger because it references line object and doesn't clean
it.
> A better check would be to run with and without the closure cleaner change
Yea, this is what I did locally, but how to write a test for 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 and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
