search

magro / kryo-serializers

More kryo serializers
Apache License 2.0
381 stars 120 forks source link

CharsetSerializer capability for kryo #44

Closed jagrutmehta closed 8 years ago

jagrutmehta commented 8 years ago

Added new serializer for UTF-8, UTF-16 and such Charsets which are not public class (either default or private/protected). Kryo deserialization fails because of this.

Added CharsetSerializer, test case and documentation.

magro commented 8 years ago

Sorry for the late response, I'm just on vacation. I'd suggest that you submit this PR directly to kryo, as we want to keep more things there and AFAICS there's nothing "special" with the suggested serializer. When submitting the PR you should submit a single commit (squash multiple commits) and don't unnecessarily change whitespaces :-)

magro commented 8 years ago

Closed for now, thanks anyways!