Closed johnynek closed 6 years ago
@ianoc can you take a look?
This is not finished, but what we have is testable and in the interest of keeping the PRs small, I'd like to merge this and then follow up with more parts:
build fails because spark is not there for 2.12. Will remove spark 2.12 from the CI.
Some comments but they are more of an ongoing discussion than anything else. LGTM
thanks for the comments. these seem like good points to keep in mind as we optimize. All of your comments are doable for sure. Will address in the follow ups.
This is a very basic beginning to a spark backend.
It is not complete, but does support map-only operations.
There is one big question: can we really just lie to spark and say we have AnyRef everywhere? I think it may just make serialization worse (kryo writing the classnames), but maybe we can circumvent that later since scalding allows configs to have registered classes named, maybe we can pass that information to spark somehow.
cc @ianoc