Skip to content
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

Support Storm's custom serialization support for multi-lang. [msgpack] #8

Closed
amontalenti opened this issue May 6, 2014 · 8 comments

Comments

@amontalenti
Copy link
Contributor

A recently merged pull request 84 on the Storm project implements custom serialization support for the multi-lang protocol. This means that you can technically register other faster serializers than JSON for Storm's tuple data, e.g. we could use msgpack.

In addition, thanks to Storm's built-in support for serialization hooks with Kyro, we could actually switch Tuple serialization from Kyro's built-in JVM-based serializer to something like msgpack. This would potentially allow us to avoid the encode/decode cost associated with using ShellBolts and ShellSpouts.

It would be great for someone from the community to do a prototype of this using a streamparse project, and prove its viability. Then we could merge the support into the core library, build a support library in Java for the custom serializers, and offer documentation for how to set up Storm to leverage them. This will likely fit in very well with our lein-based Storm project management.

@jsgilmore
Copy link

Hi, good to see someone uses my serialisation pull request. I've already implemented a protocol buffer serialization http://www.google.com/url?q=https://github.com/jsgilmore/protoshell&sa=U&ei=jDlvU4DGKsbx0gX9vYGQDQ&ved=0CAsQFjAA&usg=AFQjCNEo9x4r3QZJPuDSzv4PD-gSz0VNMA that you could also use for improved throughput.

@amontalenti
Copy link
Contributor Author

@jsgilmore Thank you. That's pretty interesting. Looks like you're using this with GoStorm?

@amontalenti
Copy link
Contributor Author

Also here are links to protoshell and gostorm for reference.

@amontalenti amontalenti added this to the v1.2 milestone Oct 14, 2014
@dan-blanchard
Copy link
Member

Now that Pyleus is out and has support for msgpack serialization, it seems like we could pretty easily adapt their code for this (since it's Apache licensed, and that should be compatible with BSD).

Here are the relevant links:

@amontalenti
Copy link
Contributor Author

Indeed, this is a good idea. Thanks for the links, @dan-blanchard.

@dan-blanchard dan-blanchard modified the milestones: v2.0, v1.2 Apr 27, 2015
@rduplain rduplain changed the title Support Storm's new custom serialization support for multi-lang Support Storm's custom serialization support for multi-lang. [msgpack] Sep 1, 2015
@rduplain
Copy link
Contributor

rduplain commented Sep 1, 2015

Related to #174.

@dan-blanchard
Copy link
Member

pystorm/pystorm#1 addresses this.

@dan-blanchard
Copy link
Member

This issue was moved to pystorm/pystorm#5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants