-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Make boilerplate syntax classes extend Serializable #3439
Make boilerplate syntax classes extend Serializable #3439
Conversation
Today I ran into a serializability issue when using `mapN` in a Spark environment 😬.
9ff69f6
to
5a0ef40
Compare
Codecov Report
@@ Coverage Diff @@
## master #3439 +/- ##
==========================================
- Coverage 91.64% 91.43% -0.21%
==========================================
Files 381 380 -1
Lines 8269 8244 -25
Branches 227 227
==========================================
- Hits 7578 7538 -40
- Misses 691 706 +15
Continue to review full report at Codecov.
|
Definite 👍 to the boilerplate additions. For |
By the way if anyone else is annoyed by scrolling past the huge Codecov comment in PRs like this, #3428 fixes that, and just needs one more review to merge. |
Pretty sure I just diagnosed the problem here. ;-) |
Merging this now. I'll include changing back the |
Today I ran into a serializability issue when using
mapN
in a Sparkenvironment 😬.