Skip to content
This repository has been archived by the owner on Apr 18, 2018. It is now read-only.

Contribute MessagePackSerializer upstream #159

Open
dan-blanchard opened this issue Oct 2, 2015 · 3 comments
Open

Contribute MessagePackSerializer upstream #159

dan-blanchard opened this issue Oct 2, 2015 · 3 comments

Comments

@dan-blanchard
Copy link

Your MessagePackSerializer class seems like it would be broadly useful in the Storm community. Would you be willing to contribute it to Apache Storm proper? It's already Apache license, and obviously so is their code.

@dan-blanchard
Copy link
Author

I don't mean to step on any toes here, but since this is so broadly useful and the licenses are compatible, I was planning on trying to get this file added upstream. I would prefer to have the 👌 from you all first, but in the end it's all just compatible open source code.

@patricklucas
Copy link
Contributor

Yes, of course! Sincere apologies for missing this question from October, thanks to @poros for pinging me about it.

I'm totally on board with sending this upstream, though I'd like to get +1s from @ecanzonieri and @poros as well.

@dan-blanchard
Copy link
Author

Coincidentally, it looks like someone has a slightly different serializer implementation they just opened a PR for, so I'm sure your input on their PR would be very valuable. It's apache/storm#1136.

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

No branches or pull requests

3 participants