Kim Gräsman
Kim Gräsman
I thought of another thing I wanted some feedback on: the basic and generic encoder/decoder pairs could benefit from being one entity, i.e Codec and GenericCodec that each can do...
Would you prefer I finish up everything in this PR, or post piecemeal PRs split into; * basic codec + soe.go utilities * generic codec * dynamic decoder with resolver,...
Actually it's probably easier to do everything together. Just worried it will turn into a megapatch, but let's worry about that if it becomes a problem.
> Bundling them seems like a good idea. Cool, done. Also added tests for generic codec, mostly repeated from basic_test.go. Still haven't finished up the DynamicDecoder + Resolver.
I think this has everything I wanted to do: * Rebased * Rename `GenericCodec` -> `TypedCodec` * Add `MemorySchemaRegistry` to use as a testing `SchemaResolver` * Finish up `DynamicDecoder` API...
Gentle Easter week ping!
Alright, squashed the history and wrote up a more proper commit message. When you're happy with this, I'd like to try it out with some of our internal code so...
Sorry for going AWOL on this PR, I ran into 6 months of unrelated chaos and still haven't had a chance to integrate this on our end. That said, it...