Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/2416#issuecomment-57072352
BTW the reason why abstract classes are favored over traits is because
traits (as with many advanced scala features) with default impls complicate a
lot of things, especially when it comes to java and binary compatibility.
I guess in this case it might be ok because Spillable is not a public thing.
---
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]
