Github user dossett commented on the pull request:
https://github.com/apache/storm/pull/1052#issuecomment-176801519
Thanks everyone for the feedback on the code. I am beginning to appreciate
some of the challenges of serialization, in particular whatever serializer is
registered with kryo can only be invoked with a default constructor, so my
naive implementation, which relies on initializations via constructors in some
spots will not work. The meaning of @revans2's comments about reading data from
a special file in the jar are also becoming clearer to me -- that's a way to
get information needed for initialization without relying on constructor
parameters. I'll take another pass at cleaning up at least some of these
issues and fixing/addressing comments. There's some excitement here about
using our Confluent registry to speed up serialization, so I really want to
make this work.
These are some new corners of java for me, so in the worst case scenario
I'll simply have learned a lot. Thanks everyone!
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
