GitHub user mateiz opened a pull request:
https://github.com/apache/spark/pull/700
SPARK-1708. Add a ClassTag on Serializer and things that depend on it
This pull request contains a rebased patch from @heathermiller
(https://github.com/heathermiller/spark/pull/1) to add ClassTags on Serializer
and types that depend on it (Broadcast and AccumulableCollection). Putting
these in the public API signatures now will allow us to use Scala Pickling for
serialization down the line without breaking binary compatibility.
One question remaining is whether we also want them on Accumulator --
Accumulator is passed as part of a bigger Task or TaskResult object via the
closure serializer so it doesn't seem super useful to add the ClassTag there.
Broadcast and AccumulableCollection in contrast were being serialized directly.
CC @rxin, @pwendell, @heathermiller
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mateiz/spark spark-1708
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/700.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #700
----
commit 9d48830dce909634b68bb6e34f928345aeab42c1
Author: Matei Zaharia <ma...@databricks.com>
Date: 2014-05-08T21:37:34Z
Add a ClassTag on Serializer and things that depend on 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
