-
Notifications
You must be signed in to change notification settings - Fork 13k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[scala] Add ClosureCleaner for Scala Lambdas #156
Conversation
Is this stable enough to go into the 0.7 release? |
I don't see any tests in the pull request. |
Yes, It would be good to have those. But it is also good to know that all the existing Scala Tests still run. |
99ad218
to
dbeface
Compare
I added some Test cases. |
dbeface
to
1f18fc7
Compare
2e0a003
to
a5cf6f5
Compare
Any comments, anyone want to do some further tests. Or should we merge this rather sooner than later. |
Looks good. Do the tests contain code from Apache Spark? If yes, I would add a note there as well... |
a5cf6f5
to
f502b5b
Compare
Ok, I added a notice to the ITCase about the origin of the tests: Spark. |
Looks good to merge... |
Manually merged. |
* [FRT-119] Removes sensitive log entries * Revert two files
No description provided.