-
Notifications
You must be signed in to change notification settings - Fork 22
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
ProtoBuf & JSON Schema support #41
Comments
Looking forward to this PR, and would love to help implement. Is there a slack/discord/forum for discussion anywhere? I was in a bit of a time crunch on a current project and did not realize this library did not support Protobuf formatted messages. In order to get around it for now we ignore SchemaRegistry, precompiled the protobufs, and then had to strip out the Magic Byte and other SchemaRegistry headers, but it does decode now. Obviously this is not ideal and kills the purpose of SchemaRegistry, so the sooner I can fix the better. I am curious how you would implement this, I am not 100% sure how the Confluent deserializers operate behind the scenes. Are they compiling the Protobufs on the fly, caching, dynamically load the classes after with reflection? |
I am planning on separating these concerns. On the HTTP side the only thing we need is a schema type and a string representation of a given schema. The hard part is that I would need to change the There are multiple options that are currently in my head: One
|
No description provided.
The text was updated successfully, but these errors were encountered: